fix #5176 "ip" text box 45 characters

e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107

https://e107.org

GNU General Public License v3.0

318 stars 212 forks source link

fix #5176 "ip" text box 45 characters #5178

Closed Jimmi08 closed 5 months ago

Jimmi08 commented 5 months ago

CLOSES #5176

Types of Changes

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Performance enhancement (non-breaking change which improves efficiency)
[ ] Code cleanup (non-breaking change which makes code smaller or more readable)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (a change to man pages or other documentation)

Checklist

[ x] My code adheres to the e107 code style standard.
[ x] I have read the contributing document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests pass.

Deltik commented 5 months ago

This test needs to be adjusted to account for the new behavior:

 ---------
1) e_formTest: Render element
 Test  tests/unit/e_formTest.php:testRenderElement
Field: ip_001
Failed asserting that two strings are equal.
- Expected | + Actual
@@ @@
-'<input type='text' name='ip_001' value='::1' maxlength='32'  id='ip-001' class='tbox form-control'  />'
+'<input type='text' name='ip_001' value='::1' maxlength='45'  id='ip-001' class='tbox form-control'  />'
#1  /__w/e107/e107/e107_tests/tests/unit/e_formTest.php:1309
#2  /__w/e107/e107/e107_tests/vendor/bin/codecept:115

Jimmi08 commented 5 months ago

@Deltik I tried, but I am not able to do more, it failed again. Freely fix it yourself.