I found a bug with internet explorer 7.0 :

When I want to apply a filter on my content under "content list", I can't !
When I choose a filter and click on "filter" button, my filter is not apply.

For information, filter work well under internet explorer without your module.

It work well with Firefox.

CommentFileSizeAuthor
#6 weight.module.patch32.96 KBtaqitaha
#3 weight.module.txt15.62 KBtaqitaha

Comments

nancydru’s picture

Assigned: Unassigned » nancydru

Well, I can reproduce this. I will try to find out why it's happening.

nancydru’s picture

It 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.

taqitaha’s picture

StatusFileSize
new15.62 KB

Hi 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 be

$selector_template = "\n"."<select style=\"margin: 0;\"
    onchange='httpRequest(\"GET\", \"" . base_path() .
    "?q=admin/node/weight/_weight_change/\" + [NID] + \"/\" +
    this.options[this.selectedIndex].value,true)' >";

so 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.

taqitaha’s picture

Status: Active » Fixed
nancydru’s picture

Status: Fixed » Needs work

An issue is not "fixed" until something is committed to CVS.

Can you create this as a patch, please?

taqitaha’s picture

StatusFileSize
new32.96 KB

sorry for that didn't know the rules
attached is a patch created using Winmerge let me know if it is ok.

taqitaha’s picture

Status: Needs work » Needs review
nancydru’s picture

Status: Needs review » Needs work

This is still a complete module replacement. I need to see what was changed.

nancydru’s picture

Status: Needs work » Fixed

Committed.

taqitaha’s picture

Please check the differences on lines 287 and 303 for changes

nancydru’s picture

Check the -dev version for what I put in.

taqitaha’s picture

Let me know where to check.

nancydru’s picture

Download the -dev version.

taqitaha’s picture

Downloaded 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?

nancydru’s picture

The -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.

Status: Fixed » Closed (fixed)

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