GeezerVisonThe World Wide Web, restored to 800 pixels of glory. |
| Home|About|Gaming cabinet|Archives|How it works|Starter sites |
GeezerVison is showing github.com — Form Validation Methods Broken by Forced Lowercasing · Issue #138 · leafsphp/docs Open DescriptionHi, There is a bug in validation that affects 'regex', 'match' and 'contains' tests. The parameters for these rules are forced to lower case so matches to fields with upper case values are not possible. As of Form 3.2 (current) in \Leaf\Form:test we see: Commenting out strtolower solves my problem, but I think it is useful since it allows rule names to be more flexible with regards to capitalization. The strtolower($currentRule) comes too early - it forces $param to lowercase since it is split from $curretnRule afterward. This breaks most useful regular expressions. I suggest moving the checks to split out parameters to immediately after $param is declared. Possibly also the previous if statement. I'm not sure how this interacts with validation of indexed arrays. It's not clear to me how parameters are handled in this case Thanks for your attention to this! ActivityMetadataMetadataAssigneesLabelsNo labels TypeProjectsMilestoneRelationshipsDevelopmentIssue actions |
| You are visitor 01402503 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |