Open Bug 509734 Opened 15 years ago Updated 11 years ago

"new Bugzilla::Bug" should return undef instead of an object with {error} set

Categories

(Bugzilla :: Creating/Changing Bugs, enhancement, P2)

enhancement

Tracking

()

People

(Reporter: mkanat, Unassigned)

References

Details

We should make Bugzilla::Bug behave consistently with all the other Bugzilla::Object implementations, and return undef instead of an object with {error} set. Then we can implement some *other* method of implementing the things that we currently implement using {error} (like just calling can_see_bug and all that directly).
This would also fix bug 348397 on tip.
Blocks: 348397
If you pass an alias which doesn't exist, new() uses this alias as bug ID. You could then have a string where a scalar is expected, which could lead to XSS if checks aren't done correctly (but probably no SQL injection as the alias is probably tainted).
Assignee: create-and-change → LpSolit
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → Bugzilla 5.0
Assignee: LpSolit → create-and-change
Target Milestone: Bugzilla 5.0 → ---
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.