523977 - Bugzilla::Object->check should trim and be more accurate about what's "empty"

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, enhancement)

Description

[Max Kanat-Alexander]

Bugzilla::Object->check currently calls trim() to check if the input value is "empty", but it doesn't pass the trimmed value to new, which it should be doing (as check() is our primary input validator, and we should be trimming values on input). Also, currently it thinks that "0" is an invalid name for an object, even though it's a perfectly valid string.

Comment 1

[Max Kanat-Alexander]

Attachment: v1

This does several things:

1) It makes the trimmed get passed to _init, which I believe we used to do before we allowed check() to take ids.

2) It makes "0" a valid id and name for check(), which means that if you pass 0 as an id, you'll get a much clearer "there is no (blah) with the id 0" instead of just "you must select/enter a (blah)".

3) I made detaint_natural and detaint_signed use int() on their return values, because that makes Perl treat the object entirely like an integer and not like a string, which could be important in certain DBDs that optimize bound values when they're integers (and also simplifies detaint_signed in any case). This also means that "000" gets detainted to "0", which is more accurate. Same for "001"--it gets detainted to "1".

Comment 2

[Frédéric Buclin]

Comment on attachment 407883 [details] [diff] [review]
v1

Looks good. r=LpSolit

Comment 3

[Max Kanat-Alexander]

Checking in Bugzilla/Object.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Object.pm,v  <--  Object.pm
new revision: 1.37; previous revision: 1.36
done
Checking in Bugzilla/Util.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Util.pm,v  <--  Util.pm
new revision: 1.94; previous revision: 1.93
done