Bug #36159

StringLengthValdiator uses strlen which is not UTF-8 save

Added by Christian Müller over 3 years ago. Updated over 2 years ago.

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

100%

Category:-
Target version:-
PHP Version: Complexity:
Has patch:No Affected Flow version:Git master

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 Müller over 2 years 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 Müller over 2 years 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 Müller over 2 years 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 Müller over 2 years 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 over 2 years 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 over 2 years 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 over 2 years 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 over 2 years 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 over 2 years 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 over 2 years 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 over 2 years 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 Müller over 2 years ago

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

#9 Updated by Gerrit Code Review over 2 years 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 Müller over 2 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF