Bug #36159

StringLengthValdiator uses strlen which is not UTF-8 save

Added by Christian Mueller about 1 year ago. Updated about 1 month ago.

Status:Resolved Start date:2012-04-16
Priority:Should have Due date:
Assignee:- % Done:

100%
Category:-
Target version:-
PHP Version: Complexity:
Has patch:No FLOW3 version affected:Git master
Votes: 0

Description

Will probably result in not correctly validated obejcts. How to abstract UTF-8 save string length functions like mb_strlen?

Associated revisions

Revision e32d5e69
Added by Christian Mueller 2 months ago

[BUGFIX] StringLengthValidator should use UTF-8 safe function

StringLengthValidator no longer uses strlen to determine the
length of the string to be validated but uses the
Utility\Unicode\Functions::strlen method which relies on
mb_string internally.

Change-Id: I3054a497173f0bd35f7054a073d90c5bdf710ea8
Fixes: #36159
Releases: master, 2.0, 1.1

Revision 33ca0190
Added by Christian Mueller 2 months ago

[BUGFIX] StringLengthValidator should use UTF-8 safe function

StringLengthValidator no longer uses strlen to determine the
length of the string to be validated but uses the
Utility\Unicode\Functions::strlen method which relies on
mb_string internally.

Change-Id: I3054a497173f0bd35f7054a073d90c5bdf710ea8
Fixes: #36159
Releases: master, 2.0, 1.1

Revision 39431120
Added by Christian Mueller 2 months ago

[BUGFIX] StringLengthValidator should use UTF-8 safe function

StringLengthValidator no longer uses strlen to determine the
length of the string to be validated but uses the
Utility\Unicode\Functions::strlen method which relies on
mb_string internally.

Change-Id: I3054a497173f0bd35f7054a073d90c5bdf710ea8
Fixes: #36159
Releases: master, 2.0, 1.1

Revision fc822cd3
Added by Christian Mueller about 1 month ago

[BUGFIX] StringLengthValidator should use UTF-8 safe function

StringLengthValidator no longer uses strlen to determine the
length of the string to be validated but uses the
Utility\Unicode\Functions::strlen method which relies on
mb_string internally.

Change-Id: I3054a497173f0bd35f7054a073d90c5bdf710ea8
Fixes: #36159
Releases: master, 2.0, 1.1

History

#1
Updated by Gerrit Code Review 3 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18384

#2
Updated by Gerrit Code Review 3 months ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18384

#3
Updated by Gerrit Code Review 3 months ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18384

#4
Updated by Gerrit Code Review 3 months ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18384

#5
Updated by Gerrit Code Review 2 months ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18384

#6
Updated by Gerrit Code Review 2 months ago

Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/19300

#7
Updated by Gerrit Code Review 2 months ago

Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at https://review.typo3.org/19301

#8
Updated by Christian Mueller 2 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#9
Updated by Gerrit Code Review about 1 month ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch composer has been pushed to the review server.
It is available at https://review.typo3.org/19380

#10
Updated by Christian Mueller about 1 month ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF