Item3638: Edit link in actionnotify mail does not work in IE7

At the office, everyone is now using IE7. I'm trying to use the actionnotify system, and it works fine, but when people click the 'edit' link in the notification mail, instead of:

http://wiki/bin/edit/Test/TestTopic?skin=action,pattern;atp_action=000075;t=1171555347

which is the link in the e-mail, the browser address is mangled to become

http://wiki/bin/edit/Test/TestTopic?skin=action

as a consequence, the topic is shown in raw mode instead of the editing of the action. When I copy/paste the link into the address bar of the browser, all works well. Unfortunately I cannot test this on IE6 anymore. Hope there's a solution.

-- TWiki:Main/JosMaccabiani - 15 Feb 2007

Unfortunately I can't test in IE7, so I'm unable to see this. However it would appear that IE7 is having problems with the comma. Comma is a special character in URLs, and should be URL encoded.

Please try this:

Take the action link you give above, and paste it into the browser link bar. Then edit the link and change the comma to %2C. Let me know if that cures the problem.

If it works, the code change necessary is this:

In Action.pm

# PRIVATE format edit field
sub _formatField_edit {
    my ( $this, $asHTML, $type, $newWindow ) = @_;

    if ( !$asHTML ) {
        # Can't edit from plain text
        return '';
    }

    my $skin = join( ',', ( 'action', TWiki::Func::getSkin()));

    my $url = TWiki::Func::getScriptUrl(
        $this->{web}, $this->{topic}, 'edit').
          '?skin='.$skin.';atp_action=' . $this->getAnchor() . ';t='.time();

Change this last bit to:

    my $url = TWiki::Func::getScriptUrl(
        $this->{web}, $this->{topic}, 'edit',  skin=>$skin, atp_action=>$this->getAnchor(), t=>time());

-- TWiki:Main.CrawfordCurrie - 18 Feb 2007

Update:

• directly pasting the URL in IE7 address bar works, even with the comma
• manually changing the comma to %2C works as well
• clicking on the 'edit' link from the mail, using Thunderbird, does not work

Thanks for the quick patch! I'm a bit shy about notifying people too often, so I won't know if it works untill Thursday, but I'll let you know.

-- TWiki:Main.JosMaccabiani - 19 Feb 2007

Yes, the patch works great!

-- TWiki:Main.JosMaccabiani - 21 Feb 2007

Thanks for testing. Updated version uploaded.

CC