Closed Bug 657034 Opened 11 years ago Closed 11 years ago

Timed Expiration queries need to bind null too

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla6

People

(Reporter: sdwilsh, Assigned: sdwilsh)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

This was missed in bug 520165.  We are binding to the wrong named parameter (it doesn't exist).
Summary: Timed Expiration doesn't bind anything to limit_uris → Timed Expiration doesn't bind anything to limit_uris and null_skips_last
I was looking at the wrong query.  We just need to bind null here.
No longer blocks: 520165
Summary: Timed Expiration doesn't bind anything to limit_uris and null_skips_last → Timed Expiration queries need to bind null too
Attached patch v1.0Splinter Review
Attachment #532364 - Flags: review?(mak77)
Comment on attachment 532364 [details] [diff] [review]
v1.0

Review of attachment 532364 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #532364 - Flags: review?(mak77) → review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/645b6e992890
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.