? Success

User tests: Successful: Unsuccessful:

avatar Radek-Suski Radek-Suski - open - 15 Nov 2013
avatar Radek-Suski
Radek-Suski - comment - 15 Nov 2013

Wow. Sorry. I didn't knew that it creates an issue report when you send a pull request
You may blame the J!Tracker Application for transmitting this comment.

avatar betweenbrain
betweenbrain - comment - 26 Nov 2013

What is the status of this PR? Any tracker item?

avatar infograf768
infograf768 - comment - 30 Nov 2013

As far as I remember, the PR/commit was made to accommodate the language string which clearly states that there should be no space in the username.

JLIB_DATABASE_ERROR_VALID_AZ09="Please enter a valid username. No spaces, at least %d characters and must <strong>not</strong> contain the following characters: < > \ "_QQ_" ' &#37; ; ( ) &"

(last characters displayed wrong as pasting here changes them)

Also, it corrects some other aspects concerning forbidden characters.
A second PR/commit followed allowing users to reset their username in their profile if there was a space in it.

avatar snaffle
snaffle - comment - 3 Dec 2013

Just want to agree that this can be a problem for existing 2.5 sites - we have 4K users, a large number of which have spaces and it broke user edit for them... had to revert the users.php file back to the 2.5.14 version in the short term.

Look forward to seeing it fixed in the next update.

avatar infograf768
infograf768 - comment - 3 Dec 2013

can we have a patch vs 2.5.x taking into account both these commits?
07-Oct-2013 Jean-Marie Simonet
# [#30586] Usernames that should not be accepted can be actualy registered. Thanks Sergio Manzi
# [#32199] *Allow username edit in profile edit when username is not compliant. Thanks Achal Aggarwal

  • a specific tracker on joomlacode please
avatar infograf768
infograf768 - comment - 4 Dec 2013
avatar infograf768
infograf768 - comment - 4 Dec 2013

#2643 // master
#2644 // 2.5x

Thanks for testing

avatar infograf768
infograf768 - comment - 4 Dec 2013

Hmm. my PRs are based on this one and this accepts spaces also at beginning and end of username.
What you think?

avatar betweenbrain
betweenbrain - comment - 5 Dec 2013

I'd not allow spaces at the beginning and end of username. I think that could cause too much potential confusion from accidents.

avatar infograf768
infograf768 - comment - 5 Dec 2013

I have modified the PRs to not allow them indeed.
Changed the Lang string to fit. Please test and comment on http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=32883&start=0

avatar infograf768
infograf768 - comment - 5 Dec 2013

new PR for master is
#2646

PR for 2.5.x unchanged #2644

avatar infograf768
infograf768 - comment - 6 Dec 2013

Closing this one as the new ones have now been commited

avatar infograf768 infograf768 - change - 6 Dec 2013
Status New Closed
Closed_Date 0000-00-00 00:00:00 2013-12-06 06:44:40
Labels Added: ?
avatar infograf768 infograf768 - close - 6 Dec 2013
avatar infograf768 infograf768 - close - 6 Dec 2013
avatar Radek-Suski Radek-Suski - reference | cc5bb94 - 1 Jan 15

Add a Comment

Login with GitHub to post a comment