Attached are patches to the install, info and module files for Drupal 6. Please review and let me know what needs improvement. I've tested flagging and unflagging nodes, comments and users, as well as reviewing flagged content. I look forward to any feedback. Hopefully the patch format is ok.

CommentFileSizeAuthor
#2 flag_content.module.patch8.22 KBcfuller12
flag_content.module.patch8.77 KBcfuller12
flag_content.install.patch3.17 KBcfuller12
flag_content.info.patch592 bytescfuller12

Comments

kbahey’s picture

kbahey commented
Status: Needs review » Needs work

Thanks a lot for this.

The patch format is OK.

However, I had a small change go in a few hours ago, so please refresh your CVS copy and reroll the patch so it does not undo that change (has to do with separating permissions).

cfuller12’s picture

cfuller12 commented
StatusFileSize
new flag_content.module.patch8.22 KB

Got it. Also realized I had a bit of debug code in there.

kbahey’s picture

kbahey commented
Status: Needs work » Fixed

The permissions were still wrong, so I fixed them.

Committed. There will be a 6.x branch soon.

Thanks a lot.

cfuller12’s picture

cfuller12 commented

Hmm, not sure why it didn't get updated, but thanks for fixing it. Let me know if there's anything else needed.

kbahey’s picture

kbahey commented

It got updated in some places, but in one place the old perm was there.

CVS/patch thinks you wanted it the other way.

Nevermind.

It is all Ok now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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