Zend Framework

Different pattern may be needed for some language in Zend_Filter_Alnum.

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: 1.0.2
  • Fix Version/s: 1.5.0
  • Component/s: Zend_Filter
  • Labels:
    None
  • Fix Version Priority:
    Should Have

Description

Similar reason as ZF_2108, I propose to update this component like as the attachment.

  1. zf_2107_20071126.txt
    26/Nov/07 4:17 AM
    0.8 kB
    Satoru Yoshida
  2. zf_2107_20071127.txt
    27/Nov/07 1:26 AM
    0.6 kB
    Satoru Yoshida
  3. ZF_2107_20071127testpattern.txt
    27/Nov/07 1:27 AM
    0.8 kB
    Satoru Yoshida

Issue Links

Activity

Hide
Satoru Yoshida added a comment -

I mistook. in the description. It should apply to Zend_Filter_Alpha.

I correct description in following:
^\p{Ll}\p{Lu}\p{N} is more better.

Show
Satoru Yoshida added a comment - I mistook. in the description. It should apply to Zend_Filter_Alpha. I correct description in following: ^\p{Ll}\p{Lu}\p{N} is more better.
Hide
Satoru Yoshida added a comment -

Please see attachment.

Show
Satoru Yoshida added a comment - Please see attachment.
Hide
Satoru Yoshida added a comment -

Code for Zend_Filter_Alnum

Show
Satoru Yoshida added a comment - Code for Zend_Filter_Alnum
Hide
Satoru Yoshida added a comment -

I mistook in attachment.
^[:alpha:]\p{N} should been ^[:alnum:]

Show
Satoru Yoshida added a comment - I mistook in attachment. ^[:alpha:]\p{N} should been ^[:alnum:]
Hide
Satoru Yoshida added a comment -

Correct about zf_2107_20071126.txt

Show
Satoru Yoshida added a comment - Correct about zf_2107_20071126.txt
Hide
Satoru Yoshida added a comment -

I make test pattern for Zend_Filter_AlnumTest.php . I hope this will be useful.

Show
Satoru Yoshida added a comment - I make test pattern for Zend_Filter_AlnumTest.php . I hope this will be useful.
Hide
Wil Sinclair added a comment -

What is the status for this issue and what action do we plan to take on it?

Show
Wil Sinclair added a comment - What is the status for this issue and what action do we plan to take on it?
Hide
Satoru Yoshida added a comment -

I offered a solution in the file attachments, but it does not be discussed or applied.

Show
Satoru Yoshida added a comment - I offered a solution in the file attachments, but it does not be discussed or applied.
Hide
Darby Felton added a comment -

Assigning to Satoru Yoshida. Please contact me directly when you are ready to commit your changes, and I'll take care of giving you SVN credentials (e.g., username and password) needed to commit successfully. Thank you!

Show
Darby Felton added a comment - Assigning to Satoru Yoshida. Please contact me directly when you are ready to commit your changes, and I'll take care of giving you SVN credentials (e.g., username and password) needed to commit successfully. Thank you!
Hide
Satoru Yoshida added a comment -

Resolved in SVN r8710

Show
Satoru Yoshida added a comment - Resolved in SVN r8710
Hide
Wil Sinclair added a comment -

Did this make it in to 1.5.0?

Show
Wil Sinclair added a comment - Did this make it in to 1.5.0?
Hide
Satoru Yoshida added a comment -

No, I thought it into 1.0.5 if 1.0.5 is scheduled. But, I find it in 1.5.0 now. So , I change list box of Fix version to 1.5.0

Show
Satoru Yoshida added a comment - No, I thought it into 1.0.5 if 1.0.5 is scheduled. But, I find it in 1.5.0 now. So , I change list box of Fix version to 1.5.0
Hide
Darby Felton added a comment -

Yes, these changes made it into 1.5.0, as you can see from the Fisheye tab (SVN r8732).

Show
Darby Felton added a comment - Yes, these changes made it into 1.5.0, as you can see from the Fisheye tab (SVN r8732).

People

Vote (2)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved:

Time Tracking

Estimated:
1d
Original Estimate - 1 day
Remaining:
1d
Remaining Estimate - 1 day
Logged:
Not Specified
Time Spent - Not Specified