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

Add button disappears #2

Open
derixithy opened this issue Nov 1, 2016 · 2 comments
Open

Add button disappears #2

derixithy opened this issue Nov 1, 2016 · 2 comments

Comments

@derixithy
Copy link

derixithy commented Nov 1, 2016

Like the discription says, the add button has disappeared from the configure page.
screenshot from 2016-11-01 11 59 47

Just to note: i use this fork https://github.com/derixithy/ProcessGeneralSettings which adds permissions to themodule, but switching back to the original (this) does not change anything.

Edit:
I'm on ProcessWire 3.0.36

@pmarki
Copy link
Collaborator

pmarki commented Nov 5, 2016

I checked it on a fresh 3.0.39 installation and there is no such a problem. Can you investigate it further and check if there is a element (maybe its override by sth) or if it's a JS issue (any error in console)?
Probably it's JS issue because JSON data field should be hidden.

@pmichaelis
Copy link

pmichaelis commented Nov 13, 2018

If you donwload the module from githib the Module folder ist named "ProcessGeneralSettings-master", but the root variable for the asset includes (in Line 45) is a fixed string

$root = $this->config->urls->siteModules . 'ProcessGeneralSettings';

The data import produces another errror on installation:

if(isset($data['settings_import']) && $data['settings_import'] != '') { $json = json_decode($data['settings_import'], true); $data['settings'] = $data['settings_import']; $data['settings_import'] = ''; }

something like this should be added:

else { $data['settings_import'] = ''; }

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

3 participants