Ticket #78 (closed defect: fixed)

Opened 6 years ago

Last modified 15 months ago

Logout link doesn't work correctly

Reported by: BrianKoontz Owned by: DotMG
Priority: normal Milestone: 1.3.1
Component: core Version: 1.1.6.1
Severity: normal Keywords: installer configuration modrewrite login base_url
Cc:

Description (last modified by BrianKoontz) (diff)

Clicking the "Change Settings/Logout" link simply takes the user to the UserSettings page, and doesn't log out the user. Here's the URL I'm getting:

<base_url>/wikka.php?wakka=UserSettings

rewrite_mode => 0 in the wikka.config.php file.

Possibly related ticket: #276

Change History

Changed 6 years ago by DbieL

Clicking the "Change Settings/Logout" link simply takes the user to the UserSettings page (which is what I would expect it to do) The problem comes when clicking on the #%Logout#% Log out button results in the following response: SpamCopWiki : UserSettings?action=logout This page doesn't exist yet. Maybe you want to create it?

Changed 6 years ago by rhys@…

I found a fix.

1. edit usersettings.php in the actions directory 2. find line 272 and look for "actions=logout" 3. there are three sets of values for this->href, the first two are blank 4. delete the first set of empty double quotes 5. replace the value of "action=logout" to "usersettings.php&action=logout" 6. save the file

Changed 6 years ago by anonymous

  • version changed from 1.1.6.0 to 1.1.6.1

My usersettings.php is just 270 lines so that fix didn't work for me. I changed line 78 in the usersettings.php file to the following code:

<td><input type="submit" value="Update Settings" /> <input type="button" value="Logout" onclick="document.location='<?php echo $this->href("", "UserSettings&action=logout", ""); ?>'" /></td>

/Johan

Changed 5 years ago by BrianKoontz

  • description modified (diff)
  • milestone set to 1.1.7

Changed 5 years ago by DarTar

  • keywords installer configuration modrewrite login base_url added

Mahefa, this should be taken care of with the new installer/base_url settings, right?

Changed 5 years ago by DotMG

Should

I'll try to remove this onclick parameter. It's possible to be unable to logout if JS is disabled.

Changed 5 years ago by DarTar

JS should be completely useless for logging out. Just a GET parameter does the job on my current install.

Changed 5 years ago by DotMG

  • owner changed from unassigned to DotMG
  • status changed from new to assigned

Changed 5 years ago by DotMG

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with the recent commits on the UserSettings.

Changed 3 years ago by DarTar

  • milestone changed from 1.2 to 1.3

Retargeting to 1.3, this ticket has already been closed in trunk, from which 1.3 will be branched. Consider backporting urgent issues to 1.2.X

Changed 15 months ago by BrianKoontz

  • milestone changed from 1.3 to 1.3.1

Updated milestone to 1.3.1

Note: See TracTickets for help on using tickets.