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

Item5443: Searches order="editby" are sorted by login ID (cUID) instead of WikiName making result look random

Item Form Data

AppliesTo: Component: Priority: CurrentState: WaitingFor: TargetRelease ReleasedIn
Engine   Urgent Closed   patch 4.2.1

Edit Form Data

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

Detail

Two things fail in TWiki 4.2 if login names are different from wiki names:

  • %SEARCH{"whatever" order="editby"}% sorts according to login names, which looks like "random"
  • %SEARCH{"TOPICINFO.*author= .%WIKINAME%." type="regex"}% should return all pages edited by the current user, but fails if there are login names which are different from wiki names.
    • The (anyway ugly) workaround %SEARCH{"TOPICINFO.*author= .%USERNAME%." type="regex"}% fails in our case where we have special characters in our login names

Both seem to be related to the fact that with 4.2, the "internal" user name as derived from the login name is being used for TWiki's system actions.

-- TWiki:Main/HaraldJoerg - 14 Mar 2008

given that %META:TOPICINFO{author="HaraldJoerg" date="1205525517" format="1.1" version="1.1"}% is the wikiname, I'm presuming that this is a side effect of the non-TWikiUserMapping you are using.

the ordering - hell, I'd call that an URGENT bug in SEARCH.

-- TWiki:Main.SvenDowideit - 31 Mar 2008

Actually, I am using TWikiUserMapping. Both effects are caused by $cfg{AllowLoginName}=1 and the fact that TWiki 4.2 will use the $cUID in META, giving things like the following examples from two of my topics:


-- TWiki:Main.HaraldJoerg - 01 Apr 2008

mmm, missing paste?

I'm presuming that what you were going to paste was something like META:TOPICINFO{author="hjoerg" da ?

-- SvenDowideit - 11 Apr 2008

Sorry. Missing paste, indeed. But apparently a complete topicinfo line will be killed upon save, regardless of verbatim?!?? So, Here you are: Two examples of my topic info, without the trailing percent:

%META:TOPICINFO{author="BaseUserMapping_333" date="1204249600" format="1.1" version="1.22"}
%META:TOPICINFO{author="haj_64MY_46DOMAIN" date="1204249544" format="1.1" version="1.1"}

-- TWiki:Main.HaraldJoerg - 11 Apr 2008

Drats, I forgot to change the status...

-- TWiki:Main.HaraldJoerg - 14 Apr 2008

Now that the META has changed from using WikiName to something derived from login ID (equal to the login ID in most cases) - how can this problem ever be resolved using a regex search?

This non-backwards compatible change should not have been implemented like this. Instead an extra field should have been added to the topic meta for the login ID.

The sorting (order = "editby") should naturally not sort by login but by WikiName. WikiNames are there because 'KennethLavrsen' is "human" and 'c12179' is not.

How do we get out of this? This is not the only open problem we have related to the user mapping changes.

-- TWiki:Main.KennethLavrsen - 16 Apr 2008

That's a tricky one. I suspect the only realistic option is to revert the "author" field to the wikiname, add a "user" field that records the canonical user name, and update the format version. Obviously quite a lot of code would have to be inspected for correct usage of the "author"/"user" field, but otherwise it should be fairly simple to fix.

-- CrawfordCurrie - 16 Apr 2008

Who is taking action on this bug?

-- TWiki:Main.KennethLavrsen - 30 Apr 2008

Crawfords suggestion is a sensible one. Needs to be tested thoroughly, out of scope for 4.2.1 release.

-- TWiki:Main.PeterThoeny - 12 May 2008

This bug should really get split in two if there is a part that is out of scope - I feel that the sorting part should be fixable in 4.2.1.

-- TWiki:Main.SvenDowideit - 12 May 2008

Yes the part of the bug report that relates to

%SEARCH{"whatever" order="editby"}% sorts according to login names, which looks like "random" should be fixed in the scope of 4.2.1.

I have a small fix for the sort part.

-- TWiki:Main.KennethLavrsen - 13 Jul 2008

Checked in. Will create a normal priority bug item for the unclosed part and set this to Waiting For Release

Changing the headline for release notes

-- TWiki:Main.KennethLavrsen - 13 Jul 2008

ItemTemplate
Summary Searches order="editby" are sorted by login ID (cUID) instead of WikiName making result look random
ReportedBy TWiki:Main.HaraldJoerg
Codebase 4.2.0, ~twiki4
SVN Range TWiki-5.0.0, Sun, 09 Mar 2008, build 16496
AppliesTo Engine
Component

Priority Urgent
CurrentState Closed
WaitingFor

Checkins TWikirev:17013 TWikirev:17014
TargetRelease patch
ReleasedIn 4.2.1
Edit | Attach | Watch | Print version | History: r16 < r15 < r14 < r13 < r12 | Backlinks | Raw View |  Raw edit | More topic actions
Topic revision: r16 - 2008-08-04 - 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