Joomla! Issue Tracker | Joomla! CMS #9079 - Class JForm. Remove redundant trim($data)

Fixed in Code Base
12 Apr 2016
Medium
Build: staging
# 9079
Diff
bertmert:patch-55

Success

Success continuous-integration/travis-ci/pr The Travis CI build passed Details

User tests: Successful: Unsuccessful:

bertmert
6 Feb 2016

Test:
See line 2069 of file (https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/form/form.php#L2069). There is already a

$data = trim($data);

Thus we don't need the hereby removed trim in line 2080 (https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/form/form.php#L2080).

63a0b07 6 Feb 2016

Remove unnecessary trim

bertmert - open - 6 Feb 2016

bertmert - change - 6 Feb 2016

Status

New

⇒

Pending

joomla-cms-bot - change - 6 Feb 2016

Labels

Added: ?

infograf768 - comment - 6 Feb 2016

OK here. Can be merged on review.

brianteeman - change - 1 Mar 2016

Category

⇒

Libraries

brianteeman - change - 9 Mar 2016

Status

Pending

⇒

Ready to Commit

brianteeman - comment - 9 Mar 2016

RTC based on @infograf768 comment

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9079.}

joomla-cms-bot - change - 9 Mar 2016

Labels

Added: ?

wilsonge - reference | 74f75e9 - 12 Apr 16

wilsonge - merge - 12 Apr 2016

wilsonge - close - 12 Apr 2016

wilsonge - change - 12 Apr 2016

Status	Ready to Commit	⇒	Fixed in Code Base
Closed_Date	0000-00-00 00:00:00	⇒	2016-04-12 22:08:06
Closed_By		⇒	wilsonge

wilsonge - close - 12 Apr 2016

wilsonge - merge - 12 Apr 2016

joomla-cms-bot - close - 12 Apr 2016

wilsonge - change - 12 Apr 2016

Milestone

Added:

joomla-cms-bot - change - 12 Apr 2016

Labels

Removed: ?

rdeutz - change - 1 May 2016

Milestone

Removed:

rdeutz - change - 1 May 2016

Milestone

Added:

rdeutz - change - 1 May 2016

Milestone

Added:

rdeutz - change - 1 May 2016

Milestone

Removed:

Add a Comment

Older
Newer

Joomla! Issue Tracker - CMS

[#9079] - Class JForm. Remove redundant trim($data)

Add a Comment