Closed (fixed)
Project:
Inline Messages
Version:
6.x-1.3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
13 Aug 2009 at 11:31 UTC
Updated:
10 Nov 2009 at 01:40 UTC
Wow, first issue in the queue! Firstly, this is a handy little module, so thanks a lot.
When you set a correct path for the ScrollTo plugin in the settings form, you may still recieve the error:
"The Inline Messages module requires the jQuery ScrollTo plugin to work"
This is because the module is always looking at the default path and isn't looking in the new location you've just set.
I've included a patch to fix this, taken against CVS HEAD.
| Comment | File | Size | Author |
|---|---|---|---|
| inline_messages_path.patch | 1.23 KB | johngriffin |
Comments
Comment #1
Phil Wolstenholme commentedThis got me too, I'm not sure how to patch so I'll wait until this makes it into a release.
Comment #2
andrewmacpherson commentedI've tested the patch - it works.
Comment #3
andrewmacpherson commentedNote that the issue only concerns the admin settings page for the Inline Messages module.
The only problem is getting an incorrect error message on the settings page, when the jquery.scrollTo.js file is correctly installed at the path specified.
The module does behave as expected, e.g. when submitting a comment without the required comment body.
Comment #4
andrewmacpherson commentedThis also affects the inline errors module.
#572846: jQuery ScrollTo plugin path not being recognised in Inline Errors module
Comment #5
andrewmacpherson commentedComment #6
john.kenney commentedinstalled patch and it worked for me. thank you!
Comment #7
jsfwd commentedThanks for finding the bug... the issue is fixed with 6.x-1.4
@andrewmacpherson Inline Messages has replaced Inline Errors so there will be no fixes to that module.
Comment #8
andrewmacpherson commented