? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
9 Oct 2014

See discussion #4489

the code to use would be:

JFilterInput::getInstance()->clean((string) $mystring, 'trim')
avatar infograf768 infograf768 - open - 9 Oct 2014
avatar jissues-bot jissues-bot - change - 9 Oct 2014
Labels Added: ?
df856cb 9 Oct 2014 avatar infograf768 cs
avatar joomdonation
joomdonation - comment - 9 Oct 2014

Maybe you can give some sample strings (which contains non-breaking spaces and double byte whitespaces) so that we can use to test this method ?

avatar infograf768
infograf768 - comment - 9 Oct 2014

Here is a string:

nonbreaking nonbreaking
multi multi

The spaces are in the middle. Just pick them and add first and last

avatar joomdonation
joomdonation - comment - 10 Oct 2014

@Test: OK. The method successfully remove normal, non-breaking and multibyte spaces before and end of a string.

avatar infograf768 infograf768 - reference | ef49dec - 10 Oct 14
avatar infograf768 infograf768 - merge - 10 Oct 2014
avatar infograf768 infograf768 - close - 10 Oct 2014
avatar infograf768 infograf768 - close - 10 Oct 2014
avatar infograf768 infograf768 - change - 10 Oct 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-10-10 06:22:56
avatar pe7er
pe7er - comment - 10 Oct 2014

I tested on staging (Joomla! 3.3.7-dev) using patch tester but issue has not been solved.

A username like " 123", that's [space] + 123 is still accepted.
and in User Manager when you edit the user details you can see the space in the form field.

avatar joomdonation
joomdonation - comment - 10 Oct 2014

@pe7er : You need to apply both this full request and #4489 before testing. Could you please try again?

avatar pe7er
pe7er - comment - 10 Oct 2014

Sorry, I did not understand that. I'll test with #4489 patch

avatar mbabker mbabker - change - 10 Oct 2014
Milestone
avatar mbabker mbabker - change - 22 Nov 2014
Milestone
avatar mbabker mbabker - change - 22 Nov 2014
Milestone Added:
avatar mbabker mbabker - change - 22 Nov 2014
Milestone Added:
avatar mbabker mbabker - change - 22 Nov 2014
Milestone
avatar infograf768 infograf768 - head_ref_deleted - 12 Nov 2015

Add a Comment

Login with GitHub to post a comment