547846 - CSAT responses broken due to hardcoded forumIDs?

Status: VERIFIED FIXED

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

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

After choosing the "Yes, this solved my problem" link on Mobile's CSAT, I'm taken to, say:

https://mobile-support-stage.mozilla.com/tiki-feedback.php?id=50&hash=49a5ab157d3b8869ab93952e425ad973&type=forum_topic&feedback=Firefox%20Support%20Forum&commentId=51

...but I just get:

Error

Permission denied you cannot view this page

(According to Cheng, this is because we hard-code our Forum IDs?)

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

https://mobile-support-stage.mozilla.com/en-US/forum/5/50#threadId51 is the "original" threadID.

Comment 2

[Laura Thomson :laura]

Duplicate of bug 547770 ?

Comment 3

[James Socol [:jsocol, :james]]

No, bug 547770 caused an error in the JS console but did nothing in the browser.

Comment 4

[Laura Thomson :laura]

BTW I get
This forum is flat and doesn't allow replies to other replies 
frmo your link in comment #1. (Possibly deleted by the new db dump on stage?)

Comment 5

[Laura Thomson :laura]

Also, I haven't been able to repro this :(

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Sorry; full STR:

1. Visit http://mobile-support-stage.mozilla.com/en-US/forum/5
2. Ask a question while logged in (preferably)
3. Click "Watch this topic" at the top right
4. Come along as another user and choose "Proposed solution to the problem"

The original poster (1-2) should now get an email with the "Yes, this solved..." and "No, ..." links.

Clicking on those will yield the error in comment 0.

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

(BTW, step 3 shouldn't be necessary for email notification, I don't think -- I thought it should be automatic; is that a config thing, Cheng?)

Comment 8

[James Socol [:jsocol, :james]]

Attachment: there was some missing data in the tiki_polls and tiki_poll_options tables

It looks like the poll data was hard coded, in a sense, and missing. This adds that data into the tables, and made the thread solutions work for me.

Comment 9

[Paul Craciunoiu [:paulc]]

Comment on attachment 432913 [details] [diff] [review]
there was some missing data in the tiki_polls and tiki_poll_options tables

WFM. I understand this was broken when going through the tiki-feedback link, not when viewing the thread itself.

Makes me wonder what other CSAT voting may be broken.

Comment 10

[James Socol [:jsocol, :james]]

This this is pure SQL, filed bug 553044 to run the patch on mobile-support-stage.

Comment 11

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; weird that you have to explicitly "Watch this topic," even when you're a logged-in user, but that's a separate (config?) issue.