Closed (fixed)
Project:
Weight
Version:
6.x-1.0-beta3
Component:
User interface
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
24 Feb 2009 at 09:56 UTC
Updated:
11 May 2009 at 13:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
nancydruWell, I can reproduce this. I will try to find out why it's happening.
Comment #2
nancydruIt turns out that something is blocking the "refine," "undo," and "reset" buttons from firing. I did discover a validation error and fixed that, but it is still not allowing those buttons to work.
Comment #3
taqitaha commentedHi NancyDru
Hope you are doing well.
I found the issue and resolved it.
The buttons "refine," "undo," and "reset" will not work as you have a form tag inside another form tag. IE does'nt like this. change lines in file weight.module
287 and 303 Remove the
<form>start tag on 287 and the close</form>tag on 303 this will resolve the filter issue.second is the problem with the ajax GET request generated when changing the weight from the listing page.
Onchange should generate the request as follows -
http://your.domain.com/drupalf6/?q=admin/node/weight/_weight_change/2/-19 instead of
http://your.domain.com/drupalf6/admin/node/weight/_weight_change/2/-19
notice the
?q=admin. so line 287 should beso finally line 287 would look as above and line 303 would look like as below
$selector_template .= '</select>';Let me know if this is fine or if there are other ways of resolving it.
Comment #4
taqitaha commentedComment #5
nancydruAn issue is not "fixed" until something is committed to CVS.
Can you create this as a patch, please?
Comment #6
taqitaha commentedsorry for that didn't know the rules
attached is a patch created using Winmerge let me know if it is ok.
Comment #7
taqitaha commentedComment #8
nancydruThis is still a complete module replacement. I need to see what was changed.
Comment #9
nancydruCommitted.
Comment #10
taqitaha commentedPlease check the differences on lines 287 and 303 for changes
Comment #11
nancydruCheck the -dev version for what I put in.
Comment #12
taqitaha commentedLet me know where to check.
Comment #13
nancydruDownload the -dev version.
Comment #14
taqitaha commentedDownloaded the -dev version and checked. changes have been made thank you. But tell me why is it still available in the Development snapshots section and not in the Releases section in the downloads page. Are new features being added to it?
Comment #15
nancydruThe -dev release is for people to test the changes before they go into an official release. I test on my system before committing it, but I can only do so much. Other people have really "creative" ways of using the module.