EDIT: Info: Fixed. Please use latest dev snapshot of Nice messages 7.x-1.x-dev. It is a fixed version which works smoothly and quiet and is hardly tested. I only didn't had time for a new rc2 branch yet.

By testing nicemessages the hard way last 2 weeks, I found some needed improvements I commit hopefully this night. + Some small patches from the other issues will be implemented.

It seems that uninstall() was somewhat broken in the last beta (code version compatibility with Drupal updates?). I made a revision of nicemessages_uninstall() with removing variables, instances and fields.

Local tests passed successfully so far.

EDIT: please run the second patch below (comment #3) and help with report.

CommentFileSizeAuthor
#2 nicemessages_install-1279686-2.patch1.33 KBdqd
#1 nicemessages_uninstall-1279686-1.patch1.55 KBdqd

Comments

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
StatusFileSize
new nicemessages_uninstall-1279686-1.patch1.55 KB

EDIT: Don't use that patch No.1 here ! Got an empty patch file with --help info from git diff.

Hold on for next patch please ...

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
StatusFileSize
new nicemessages_install-1279686-2.patch1.33 KB

Ok, here it comes - the right one. Please run this patch 2 before uninstalling the module. I commit it on new branch as soon as others report that the patch is working.

Please try to install, make settings on admin>configuration>system and enable it in profile. Then try to disable the module and uninstall it afterwards. (Additionally - if you feel comfortable with, watch phpMyAdmin for the field_data_user_nidemessages_enabled, the corresponding revision row below and the nicemessages entries in variables.) Then, finally reinstall it.

Let me know if it works now.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Priority: Normal » Major
Status: Active » Needs review
BrittaL’s picture

BrittaL commented
Status: Needs review » Reviewed & tested by the community

Thanks for that -- digidog. I did have made testing as you described above: NO errors while reinstalling. uninstall() seems to fetch ALL nicemessages data now (unlike with many other modules where they don't)!

Thank you!

BenK’s picture

BenK commented

Subscribing

BenK’s picture

BenK commented
Issue summary: View changes

typo and patch info

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Title: Uninstalling and reinstalling causes error "database field already exist" » Uninstalling and reinstalling test reveals work to do (done!)

EDIT: Info: Fixed. Please use latest dev snapshot of Nice messages 7.x-1.x-dev. It is a fixed version which works smoothly and quiet and is hardly tested. I only didn't had time for a new rc2 branch yet.

Macronomicus’s picture

Macronomicus commented

neither one of the latest releases work for me.... the previous ones did. I dont get any errors, and the jgrowl.js and css are loading in the page... but messages are still loading in the regular way.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented

did you enabled it in your user profile and cleared all chaches ?

Macronomicus’s picture

Macronomicus commented

Yes, I also manually cleaned out the db of all nicemessages leftovers and did a clean install with the same result. I'll test it again soon to be sure and look harder for errors... there were no apparent errors in firebug though so I dunno.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented

hm ... will follow retesting ...

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Issue summary: View changes

new info

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Issue summary: View changes
Priority: Major » Normal
Status: Reviewed & tested by the community » Closed (fixed)