Closed (fixed)
Project:
Flag content
Version:
5.x-2.8
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Sep 2007 at 19:58 UTC
Updated:
26 Mar 2008 at 21:21 UTC
Jump to comment: Most recent file
Comments
Comment #1
eojthebraveOops, diffed in the wrong direction. This patch should be correct.
Comment #2
eojthebraveAnd some more tweaks
Comment #3
underpressure commentedsubscribing
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.
Comment #4
dharamgollapudi commentedSubscribing...
Comment #5
dries arnoldsThis would be very welcome indeed. Saves the need to guess why a content type or comment is flagged.
Comment #6
lennart commentedVery useful.
Comment #7
Fanaile commentedsubscribing
Comment #8
kbahey commentedThank you.
Committed to 2.7
Comment #9
Fanaile commentedHi,
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?
Comment #10
kbahey commented@eojthebrave
Can you please look into this and see what your patch did/did not do?
Comment #11
Art Morgan commentedRegarding 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.
Comment #12
eojthebraveIf 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.
Comment #13
kbahey commented@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.
Comment #14
Fanaile commented@ 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!
Comment #15
kbahey commentedPlease 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?
Comment #16
Fanaile commentedSure;
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.
Comment #17
eojthebraveSo 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.
Comment #18
Fanaile commentedYes, everything's working for me as well with 2.8 version.
Thanks so much!
Comment #19
kbahey commentedComment #20
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.