? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
20 Oct 2014

See #4838 and #4727

avatar infograf768 infograf768 - open - 20 Oct 2014
avatar jissues-bot jissues-bot - change - 20 Oct 2014
Labels Added: ?
avatar infograf768
infograf768 - comment - 20 Oct 2014

For an unknown reason (unknown to me...) this works fine except if duplicating the "Guest" viewlevel

d137881 20 Oct 2014 avatar infograf768 oops
avatar infograf768 infograf768 - change - 20 Oct 2014
Category Libraries
avatar infograf768 infograf768 - change - 20 Oct 2014
Milestone Added:
Easy No Yes
avatar joomdonation
joomdonation - comment - 20 Oct 2014

For an unknown reason (unknown to me...) this works fine except if duplicating the "Guest" viewlevel

Could you please explain more details about it ? This PR fix that issue ? Or we will have to check the issue with "Guest" view level ?

avatar infograf768
infograf768 - comment - 20 Oct 2014

Apparently, "Guest" is a very special level and it can be duplicated, although this PR clearly prevents it as it checks for the same title in the table.
The issue is that you can also delete/modify that Guest level and that, as far as I know, it is necessary to keep it on Joomla as is by default.

avatar joomdonation
joomdonation - comment - 20 Oct 2014

Agree that "Guest" is a special access level. However, I think if we delete it by mistake, we can re-create it. Just names it "Guest" and choose the "Guest" group for that access level and it still works?

avatar Bakual
Bakual - comment - 20 Oct 2014

It shouldn't matter if it's a guest viewlevel or not. You should be able to delete it or set a different one as guest without issues. It is just a setting in the user manager options which points to a certain viewlevel.

However I haven't tested if you really can change it ;)

avatar infograf768
infograf768 - comment - 20 Oct 2014

Test this PR and you will see that we can duplicate it with the same name.
Which we can't do with a French distro not using "Guest" but "Invité" for the equivalent default view level.

I can update the PR to add a conditional
if ($db->loadResult() > 0 || $this->title == "Guest")

I tested in a default install to edit Guest and change to another name.
It saves OK.
Then I edited it again to put it back to guest.
It will not save anymore with the conditional above.

avatar infograf768
infograf768 - comment - 20 Oct 2014

Hmm, could have been an issue here locally

Please test PR as is with a default Guest

avatar infograf768
infograf768 - comment - 21 Oct 2014

I confirm: was a corruption on my test site. This should be OK.

avatar fontanil
fontanil - comment - 21 Oct 2014

Trying to add a new level "Guest" : "Save failed with the following error: Level with the name "Guest" already exists." (same message with others previous levels): patch OK for me

avatar infograf768
infograf768 - comment - 21 Oct 2014

this should go into 3.4-dev

avatar zero-24 zero-24 - alter_testresult - 21 Oct 2014 - fontanil: Tested successfully
avatar wilsonge
wilsonge - comment - 21 Oct 2014

@test get sql error showing before when duplicate name. afterwards got language string explaining cause of issue. result successful

avatar wilsonge wilsonge - test_item - 21 Oct 2014 - Tested successfully
avatar wilsonge wilsonge - change - 21 Oct 2014
Status Pending Ready to Commit
avatar wilsonge
wilsonge - comment - 21 Oct 2014

RTC

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4858.

avatar infograf768
infograf768 - comment - 21 Oct 2014

Merged into 3.4.0-dev
bbad053

avatar infograf768 infograf768 - close - 21 Oct 2014
avatar infograf768 infograf768 - change - 21 Oct 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-10-21 14:56:55

Add a Comment

Login with GitHub to post a comment