Thomas Jakobi

Results 144 comments of Thomas Jakobi

Would it i.e. help to make the properties in these lines configurable by a system setting? https://github.com/modxcms/Login/blob/master/core/components/login/controllers/web/ConfirmRegister.php#L130-L131 Sorry, but I don't have any experience in balanced environments.

Maybe you could ask @opengeek wether he has some tip for this.

There is only a toPlaceholder in Login and ActiveUsers controller.

Should be an error in the docs and has to be changed there. Could you do that?

@janwidmer: The processors path should point to the processors folder and the action needs no .class suffix: ``` switch ($modx->event->name) { case 'OnSiteRefresh': $partition = 'partition'; $response = $modx->runProcessor('cache/partition/refresh', array(...

There is an exclude-tables option in https://github.com/ifsnop/mysqldump-php#dump-settings that could be added as system setting in future versions.

This has to be changed in this code part now: https://github.com/jdaehne/BackupMODX/blob/master/core/components/backupmodx/src/BackupMODX.php#L193-L202

Is this issue still valid with 3.0.x?

You have to detect in which plugin the code `\x90fI\x02\x00\x00` is used. The error could be caused by a wrong file charset that does not match the database charset.

There is an include-tables option in https://github.com/ifsnop/mysqldump-php#dump-settings that could be added as system setting future versions.