When I enable the latest module pulled from git and then go to configure it the page errors out with Error "Not Found" and the settings are never saved.
The issue lies in the validation code that tries to connect to Convio API in a way that assumes that variables are already set. However, at the point of validation they are not. Patch for the issue should be forthcoming.

CommentFileSizeAuthor
#4 validation-fails-2312771-4.patch893 bytesmakbeta
#1 validation-fails-2312771.patch4.64 KBmakbeta

Comments

makbeta’s picture

makbeta commented
StatusFileSize
new validation-fails-2312771.patch4.64 KB

Created a patch for the issue, see attached.
Can't push up due to permission issue, but will be happy to for others to reuse the more functional code.

  • asherry committed e5645a0 on 7.x-1.x authored by makbeta 
    Issue #2312771 by makbeta: Fixed Settings page cannot be saved.
asherry’s picture

asherry commented
Status: Active » Fixed

Thank you very much, great job with the patch! I had no idea people were going to download the module so fast. I've actually been working on it offline quite a bit, but I only took over the project a few weeks ago.

I did my best to merge in your changes to what I had already, please check it over again and feel free to either start a new issue on any errors or reply to this one. I have all notifications on now.

Thanks again!

makbeta’s picture

makbeta commented
StatusFileSize
new validation-fails-2312771-4.patch893 bytes

Almost there, found another small bug where the test should be reversed for validation to succeed, patch attached.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.