Bug: register clobbers TWikiUsers if the user doesn't have read access to that topic

Here's the fix:

*** User.pm     Mon Jun 21 14:18:56 2004
--- User.pm.old Thu Feb 26 17:52:37 2004
***************
*** 233,239 ****
      my ( $wikiName, $remoteUser ) = @_;
      my $today = &TWiki::formatTime(time(), "\$day \$mon \$year", "gmtime");
      my $topicName = $TWiki::wikiUsersTopicname;
!     my( $meta, $text )  = &TWiki::Store::readTopic( $TWiki::mainWebname, $topicName, 1 );
      my $result = "";
      my $status = "0";
      my $line = "";
--- 233,239 ----
      my ( $wikiName, $remoteUser ) = @_;
      my $today = &TWiki::formatTime(time(), "\$day \$mon \$year", "gmtime");
      my $topicName = $TWiki::wikiUsersTopicname;
!     my( $meta, $text )  = &TWiki::Store::readTopic( $TWiki::mainWebname, $topicName );
      my $result = "";
      my $status = "0";
      my $line = "";

Test case

deny read access for (default) users to the TWikiUsers topic, then register a new user. the topic will be replaced with: No read access ...

Environment

-- JonathanGraehl - 21 Jun 2004

Follow up

for some reason this has already been applied to CairoRelease. (probaly due to some other refactoring). Thankyou for the report. at some stage these BugReports should be made into a suite of tests.

-- SvenDowideit - 27 Jun 2004

I duplicated the fix as part of VariousAuthBugsFixed - sorry

-- JG

Fix record

this patch applies to the BejingRelease.