Closed Bug 517288 Opened 15 years ago Closed 15 years ago

parentID can be null or missing

Categories

(support.mozilla.org :: Forum, task)

task
Not set
major

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stephend, Assigned: jsocol)

Details

(Whiteboard: aaq sumo_only)

Attachments

(1 file, 1 obsolete file)

I've found a case where parentID can be missing or null, as seen here:

http://screencast.com/t/CFetQ9lw4

Wanted to get the bug filed -- I'll back-fill the steps to reproduce next.
STR:

1. Load https://support-stage.mozilla.org/tiki-view_forum.php?locale=en-US&forumId=1
2. Click "Ask a new question"
3. Choose "Problems with settings, bookmarks, passwords or other data stored by Firefox" and click "Next step"
4. Type just a space and click "Next step"
5. Type the captch, click "Next step"
6. Advance through the warnings until you see "Oops! A similar thread already exists!", and then click on "Click here to see it" (https://support-stage.mozilla.org/tiki-view_forum_thread.php?forumId=1&comments_parentId=)
Assignee: nobody → james
Whiteboard: aaq
Attached patch patch, v1 (obsolete) — Splinter Review
Looks like just a missing line break.
Attachment #401994 - Flags: review?(laura)
Attachment #401994 - Flags: review?(laura) → review+
r52132 / r52133
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
I can no longer reproduce the "Oops!  A similar thread already exists!" error message (which is a necessary precondition), so I can't verify this change.
I can reproduce the similar thread error, but the output doesn't seem to be consistent.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached patch patch, v2Splinter Review
I'm not entirely sure why the previous patch ever worked for me? This one works for me. Of course, I'm also pretty sure the last one did before I committed it, so...
Attachment #401994 - Attachment is obsolete: true
Attachment #402237 - Flags: review?(laura)
Comment on attachment 402237 [details] [diff] [review]
patch, v2

It worked for me too - you are not going crazy.  Maybe we backed something out that re-broke it?  But this works.
Attachment #402237 - Flags: review?(laura) → review+
r52193 / r52194
Status: REOPENED → RESOLVED
Closed: 15 years ago15 years ago
Resolution: --- → FIXED
Verified FIXED; reproduced the "duplicate/similar thread exists" error, clicked through, and no longer see this error.
Status: RESOLVED → VERIFIED
Whiteboard: aaq → aaq sumo_only
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: