I needed to allow users to input their name, email address, and a reason when flagging a page. This patch adds name, email, and reason fields to the form when you click the flag link on a node. If a user is logged in it will pre-populate the name, and and email fields for you.

This new information is also sent in the e-mail to the admin, and shown in the table when the admin is viewing flagged nodes.

Comments

eojthebrave’s picture

StatusFileSize
new4.76 KB

Oops, diffed in the wrong direction. This patch should be correct.

eojthebrave’s picture

StatusFileSize
new5.61 KB

And some more tweaks

underpressure’s picture

subscribing

any idea when this feature will be implemented?

hope you could leave a custom description under the reason for submitting field, example:

reason for submitting, e.g: misspelled work, abusive, etc.

dharamgollapudi’s picture

Subscribing...

dries arnolds’s picture

This would be very welcome indeed. Saves the need to guess why a content type or comment is flagged.

lennart’s picture

Very useful.

Fanaile’s picture

subscribing

kbahey’s picture

Status: Needs review » Fixed

Thank you.

Committed to 2.7

Fanaile’s picture

Hi,

I went through today and tried to install this patch, and have run into a couple of problems. I'm not sure what's causing the problems, everything looks right, and the database looks like it's pulling from the patch. The site shows that everything is correct in the settings, and I'm not receiving any error messages. I went through and ran update.php, and cleared my cache. But this doesn't seem to be working.

There is no change in the function of the flag content operation. There is no "reason" field offered, and it is not pulling their email addresses.

One thing that did change when I applied this patch was the links in the emails that get sent to admin (which I wasn't really trying to fix at the time, but Bonus!, LOL). The links in the email were originally reading http://www.progressiveu.orghttp://www.progressiveu.org/node... - now they are reading correctly and we are able to click through to the node being flagged.

Any ideas?

kbahey’s picture

Version: 5.x-2.5 » 5.x-2.7
Status: Fixed » Active

@eojthebrave

Can you please look into this and see what your patch did/did not do?

Art Morgan’s picture

Regarding Fanaile's comment #9, we finally got it working. It was was a bit tricky though. The patch installed fine, but even after uninstalling and installing the module it still seemed to be using the same old code. Then, after a few hours and a couple more uninstall reinstalls it started partially working. However, the install script did not add the new columns to the database, so I had to add them manually. After that everything seems to work.

Would the code for the module have been cached in the database? Or does Apache cache that somewhere? That's the only reason I can think of why it might not have worked for the first few hours.

eojthebrave’s picture

If you are still having trouble with it let me know. However, I would suggest just downloading and using the latest version of the module which seems to have this rolled into it.

kbahey’s picture

Version: 5.x-2.7 » 5.x-2.8

@Art

Can you please try 2.8? It has some database schema fixes.

Please uninstall and then reinstall the new version.

If it does not work, please post here.

@eojthebrave

Please test it as well and let me know if you have issues.

Fanaile’s picture

@ kbahey and eojthe brave

Hi;

Art and I have installed the latest patch. The first few hours were a bit rocky, in that we had to uninstall and reinstall several times to try to get it to work and we weren't sure why the changes weren't becoming apparent. But everything seems to be working out well now. Art tested it quite a bit last night and I've gone through this morning and everything is working the way it's supposed to.

Thank you very much!

kbahey’s picture

Please do not use the patch. The patch is included in 2.8.

We need to test that version so as to know if there are still issues with the official tar ball or not.

Can you uninstall it, remove the module's directory and install the tarball and then install it again?

Fanaile’s picture

Sure;

I will let you know how it works out. It might take a little while for me to get to (there's a lot on my to-do list today) but I'll be sure to let you know how/if the newest version is working.

eojthebrave’s picture

So far seems to be working as expected for me. Installation went without a hitch, and was able to submit a few "reports" without any problems.

Fanaile’s picture

Yes, everything's working for me as well with 2.8 version.

Thanks so much!

kbahey’s picture

Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.