tobybatch / ChromeKimaiBridge

Allows time to logged from ChromePlugin tickets into a Kimai instance
MIT License
12 stars 2 forks source link

unexpected TimesheetService #13

Open mediaformat opened 3 years ago

mediaformat commented 3 years ago

Tried to install, getting this error: php.CRITICAL: Uncaught Error: syntax error, unexpected 'TimesheetService' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) {"exception":"[object] (ParseError(code: 0): syntax error, unexpected 'TimesheetService' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) at /var/www/kimai2/var/plugins/ChromePluginBundle/Controller/ChromeController.php:44)"} []

it looks like my build is using symfony/monolog-bundle v3.5.0

tobybatch commented 3 years ago

Hi what version of kimai are you running against? I'm currently re-writing the while extension as a raect app and kimai has had a few releases since then.

Do you fancy testing the new app? it's nearly ready for alpha.

mediaformat commented 3 years ago

I'm running v1.11 stable

When trying kimai:reload I get the following:

php.CRITICAL: Uncaught Error: syntax error, unexpected 'EntityManagerInterface' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) {"exception":"[object] (ParseError(code: 0): syntax error, unexpected 'EntityManagerInterface' (T_STRING), expecting function (T_FUNCTION) or const (T_CONST) at /var/www/kimai2/var/plugins/ChromePluginBundle/Controller/BaseController.php:26)"} [] [2021-01-04 12:53:10] php.INFO: User Deprecated: The "excluded_404s" option is deprecated in MonologBundle since version 3.4.0, you should rely on the "excluded_http_codes" option instead. {"exception":"[object] (ErrorException(code: 0): User Deprecated: The \"excluded_404s\" option is deprecated in MonologBundle since version 3.4.0, you should rely on the \"excluded_http_codes\" option instead. at /var/www/kimai2/vendor/symfony/monolog-bundle/DependencyInjection/MonologExtension.php:388)"} []

tobybatch commented 3 years ago

Which branch are you on? Can you let me know which commit you are on (git rev-parse HEAD)?

On master that file does not exist.

mediaformat commented 3 years ago

The 1st error was from Master, the 2nd was from 12_refactor_as_react_app from this repo.

tobybatch commented 3 years ago

The second on will never work with the existing plugin, it's a total refactor. :smile:

which version of PHP are you using? It needs to be php7.4 or above

mediaformat commented 3 years ago

Unfortunately it might take a while before I can install php7.4 So the issue can probably be closed for now