Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor errors in formalicious.class.php #28

Open
patrickatwsrn opened this issue May 10, 2023 · 3 comments
Open

Minor errors in formalicious.class.php #28

patrickatwsrn opened this issue May 10, 2023 · 3 comments

Comments

@patrickatwsrn
Copy link

While tracking down an issue in formalicious (#27), I found this inside my modx error log:

[2023-05-10 10:07:55] (ERROR @ /var/www/vhosts/*******/httpdocs/modx/core/components/formalicious/model/formalicious/formalicious.class.php : 122) PHP warning: Undefined array key "branding_url_help"
[2023-05-10 10:07:55] (ERROR @ /var/www/vhosts/*******/httpdocs/modx/core/components/formalicious/model/formalicious/formalicious.class.php : 122) PHP warning: Undefined array key "branding_url_help"
[2023-05-10 10:07:55] (ERROR @ /var/www/vhosts/*******/httpdocs/modx/core/components/formalicious/model/formalicious/formalicious.class.php : 220) PHP warning: Undefined array key "corePath"
@patrickatwsrn
Copy link
Author

2do: New issue for wrong version number in core/components/formalicious/model/formalicious/formalicious.class.php

Shouldn't this be 2.0.5 instead of 2.0.3 ?

@patrickatwsrn
Copy link
Author

https://github.com/Sterc/Formalicious/blob/8ebad78bc9d1e3fa7847161347066ab500a2d3c0/core/components/formalicious/model/formalicious/formalicious.class.php

2do branding_help_url !== branding_url_help

Line 55 sets the key to branding_help_url

'branding_help_url' => $this->modx->getOption('formalicious.branding_url_help'),

BUT line 85 asks for branding_url_help

$url = $this->getOption('branding_url_help');

@patrickatwsrn
Copy link
Author

And another one:

Line 40 creates a key for core_path

'core_path' => $corePath,

BUT line 220 is looking for corePath

$pubKeyFile = $this->config['corePath'] . '.pubkey';

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant