? Success

User tests: Successful: Unsuccessful:

avatar compojoom
compojoom
15 Mar 2015

...validation, not JS!).

This is the same fix as here: #3026, except that in the old pull request I've somehow screwed it up...

When I originally fixed this issue, the configuration didn't have Javascript validation. That is why I was able to run into this. Right now the config has a javascript validation and we can only run into the issue when we turn it off. But despite the fact that we can't run into the issue when js validation is on, it doesn't change the fact that the php code is wrong.

Testing instructions:
First we need to disable the javascript validation. Go to:
administrator\components\com_config\view\component\tmpl\default.php
comment out line 23,24 & 27. At the end your code should look like this:

Joomla.submitbutton = function(task)
    {
//      if (task === "config.cancel.component" || document.formvalidator.isValid(document.getElementById("component-form")))
//      {
            Joomla.submitform(task, document.getElementById("component-form"));
//      }
    };

After that test it with CMandrill:
First download CMandrill from:
https://compojoom.com/downloads/official-releases-stable/mandrill

then install it. Now go to components -> CMandrill options. Go to the permissions tab and try to change any permission. Save. The page will refresh without any error message and the permissions that you've applied shouldn't be saved.

Now apply the patch and try to save the config again. This time you'll see the error message.

avatar compojoom compojoom - open - 15 Mar 2015
avatar joomla-cms-bot joomla-cms-bot - change - 15 Mar 2015
Labels Added: ?
avatar compojoom
compojoom - comment - 15 Mar 2015

will do a new pull request as this one mixes 2 different pull requests.

avatar compojoom compojoom - change - 15 Mar 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-03-15 13:57:58
avatar compojoom compojoom - close - 15 Mar 2015

Add a Comment

Login with GitHub to post a comment