Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
forms system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
11 Apr 2012 at 10:43 UTC
Updated:
4 Jan 2014 at 01:39 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #0.0
Niklas Fiekas commentedUpdating introduction.
Comment #0.1
Niklas Fiekas commentedMake a table.
Comment #1
Niklas Fiekas commentedFirst failing tests for user_picture_file_size, filter_url_length and minimum_word_size.
Comment #1.0
Niklas Fiekas commentedimage_jpeg_quality is properly validated.
Comment #1.1
Niklas Fiekas commentedNew tests created.
Comment #3
Niklas Fiekas commentedThat + attempts to add validation.
Comment #5
Niklas Fiekas commentedConvert the other occurences.
I am not sure why #3 is still failing, that is: The user configuration form doesn't save invalid input, but it says it does and it also doesn't show validation errors.
Comment #7
Niklas Fiekas commentedOk, typo there.
This should do it.
Comment #8
aspilicious commentedHow can you leave this blank when the minimum value is 1...
Comment #9
Niklas Fiekas commentedJust leave it blank ;)
Seriously: The number will only be validated if it is not
''. So either valid or blank are allowed, unless #required is set.Comment #10
aspilicious commentedI wrote part of that function
#shame
Comment #11
cosmicdreams commentedRead through patch on #7 but I'm still confused: What's left to do after #7's patch
Comment #12
Niklas Fiekas commentedIf everything that's there is OK, that's it.
Comment #13
ericduran commentedThis is awesome. :)
Comment #14
Niklas Fiekas commentedRerolling because of #1299424: Allow one module per directory and move system tests to core/modules/system.
Comment #15
bleen commentedRTBC
Comment #16
Niklas Fiekas commentedThank you for the testing, bleen18.
Comment #17
Niklas Fiekas commentedNot to forget: The ones with the ! need to be backported to D7, though we cant use the shiny number element there.
Comment #18
dries commentedGreat job. Nice clean-up. Committed to 8.x.
Comment #19
aspilicious commentedBack to needs work see #17
Comment #20
Niklas Fiekas commentedFortunately only the simple cases were missing validation. That is only integers that must be positive.
Comment #22
Niklas Fiekas commentedNext try.
Edit: OK. I believe this is ready.
Comment #23
xjmThe backport looks complete to me. Can't think of a reason not to backport it.
Comment #24
webchickWow, nicely spotted!
Committed and pushed to 7.x. Thanks!
Comment #25.0
(not verified) commentedFactor out file field to a seperate issue.