• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

Item5091: User mapping does not map login to wikiname on old revisions of topics when using HistoryPlugin

Item Form Data

AppliesTo: Component: Priority: CurrentState: WaitingFor: TargetRelease ReleasedIn
Engine   Urgent Closed   minor 4.2.0

Edit Form Data

Summary:
Reported By:
Codebase:
Applies To:
Component:
Priority:
Current State:
Waiting For:
Target Release:
Released In:
 

Detail

It seems the last changed to the user mapping code has introduced a new bug

When you look at earlier revisions of topics you cannot see who edited them

At least not in the configuration where you map login ID to wikinames.

The strange thing is that the user is shown at the bottom of the topic but not at the top and not in the revision history you see in the history plugin.

Examples

http://merlin.lavrsen.dk/twiki42/bin/view/Myweb/WebHome?rev=8

http://merlin.lavrsen.dk/twiki42/bin/oops/Myweb/WebHome?template=oopshistory

-- TWiki:Main/KennethLavrsen - 06 Dec 2007

In the error log I have lots of

[Thu Dec 06 23:34:37 2007] [error] [client 124.168.137.74] Use of uninitialized value in string eq at /usr/local/apache2/twiki42/lib/TWiki/Users/TWikiUserMapping.pm line 409., referer: http://develop.twiki.org/~twiki4/cgi-bin/view/Bugs/Item5091 [Thu Dec 06 23:34:37 2007] [error] [client 124.168.137.74] Use of uninitialized value in string eq at /usr/local/apache2/twiki42/lib/TWiki/Users/TWikiUserMapping.pm line 512., referer: http://develop.twiki.org/~twiki4/cgi-bin/view/Bugs/Item5091

which may explain things

-- TWiki:Main.KennethLavrsen - 06 Dec 2007

pretty much another make the code do what the docco says, and write the docco job.

-- SvenDowideit - 07 Dec 2007

I have to reopen this one

After I have installed the 4.2.0 RC2 at Motorola all our history shows the editor of all revisions generated before the 4.2.0 RC2 as a link to the wikiname in the current web instead of the Main web.

The reason is that in the old days 4.1.2 and earlier it was the wikiname that was stored in the ,v files. Now it is the login name. So all revisions not being the login name are not mapped to the wikiname and we end up with wrong links to wikinames.

Sven, you need to give this another go.

I have confirmed that the problem is still there in the current code.

-- KennethLavrsen - 15 Dec 2007

I have uploaded http://mail.twiki.org/pub/Bugs/Item5091/TestTopic19.zip with a file that shows the problem. To see the problem add a line in TWikiUsers that says

   * KennethLavrsen - c12179 - 01 Oct 2007

and make a topic in Main called KennethLavrsen

And naturally run TWiki with ApacheLogin and no password manager.

-- TWiki:Main.KennethLavrsen - 15 Dec 2007

Actually this one is not really a user mapping error.

TWiki::Func::userToWikiName has a 2nd parameter called $dontAddWeb.

When the mapper finds the user it prefix correctly with Main.

But when it cannot find the user it does not prefix the login ID with Main.

But if the plugin author requested the web name to be included then Func should add it no matter what. This way old topics where the wikiname is stored instead of the login ID will still be reported correctly in plugins such as HistoryPlugin.

Unless someone objects I can fix this in func.

-- TWiki:Main.KennethLavrsen - 17 Dec 2007

I convinced myself that the fix is right. Unittests pass.

-- TWiki:Main.KennethLavrsen - 17 Dec 2007

ItemTemplate
Summary User mapping does not map login to wikiname on old revisions of topics when using HistoryPlugin
ReportedBy TWiki:Main.KennethLavrsen
Codebase 4.2.0, ~twiki4
SVN Range TWiki-4.3.0, Sun, 25 Nov 2007, build 15790
AppliesTo Engine
Component

Priority Urgent
CurrentState Closed
WaitingFor

Checkins TWikirev:15914 TWikirev:15915 TWikirev:15916 TWikirev:15917 TWikirev:16012 TWikirev:16013
TargetRelease minor
ReleasedIn 4.2.0
Topic attachments
I Attachment History Action Size Date Who Comment
Compressed Zip archivezip TestTopic19.zip r1 manage 5.6 K 2007-12-15 - 21:34 UnknownUser Test topic that has old versions with Wikiname and newest is with login
Texttxt WebHome.txt r1 manage 2.5 K 2007-12-06 - 23:01 UnknownUser  
Unknown file formattxtcommav WebHome.txtCOMMAv r1 manage 4.9 K 2007-12-06 - 23:03 UnknownUser  
Edit | Attach | Watch | Print version | History: r12 < r11 < r10 < r9 < r8 | Backlinks | Raw View |  Raw edit | More topic actions
Topic revision: r12 - 2007-12-17 - KennethLavrsen
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2024 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback