Hi,

We need to have users work on nodes for a while, and only submit them for approval when they are ready. Modr8 is a great module, but does not support draft versioning.

Can this be added as future enhancement?

Alternatively, make it usable with the new module called "save as draft". This module adds a checkmark to the input form, which enables a user to save the node as a draft. This ensures the node from being published and stay unpublished as long as it is a draft.

However, Modr8 also shows unpublished nodes in the moderation queue making it virtually impossible to distinguish what users are working on and what is ready for approval. I believe this was changed in version 5.xx, but is it possible to add this option to the content type or block?

CommentFileSizeAuthor
#10 hide-unpublished-230270-10-5x.patch4.33 KBpwolanin
#3 hide-unpublished-230270-2-5x.patch2.02 KBpwolanin
#2 hide-unpublished-230270-2.patch1.73 KBpwolanin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

Hmm, that probably isn't too hard to add back- just look at the query

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Active » Needs review
FileSize
hide-unpublished-230270-2.patch1.73 KB

please test this patch and tell me if it works. The patch is for 6.x.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
FileSize
hide-unpublished-230270-2-5x.patch2.02 KB

here's a 5.x version of the patch

storbaek’s picture

storbaek CreditAttribution: storbaek commented

Thanks pwolanin. Is it possible to release it in the .dev version? I haven't patched anything before. Alternatively, if drupal has a good how-to on patching, I can try this.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

there's a tutorial here: http://drupal.org/patch

depending on your platform, you may need to go get some software. Also, for a small patch you can apply the patch by hand - add the lines starting with +, and remove those starting with -

storbaek’s picture

storbaek CreditAttribution: storbaek commented

Ok, managed to patch it and it seems to be working correctly!

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Needs review » Needs work

I think the patch above missed the block code - I started working on a new patch in Boston, but I haven't gotten back to it.

storbaek’s picture

storbaek CreditAttribution: storbaek commented

Hi Peter

Anything ready for test?

Thanks in advance,
Dan

storbaek’s picture

storbaek CreditAttribution: storbaek commented
Priority: Normal » Critical

Any timeline on the update? We really love the module, but need this update when tested and complete :-)

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Needs work » Needs review
FileSize
hide-unpublished-230270-10-5x.patch4.33 KB

Note - there is a critical modr8 bug here: http://drupal.org/node/230033

please test that patch, since I'm not going to put this chnage into a release without that fix

storbaek’s picture

storbaek CreditAttribution: storbaek commented

Just tried several times to patch both patches but unsuccessfull. I keep recieving the error that it failed. I use cygwin and in c:\patchfiles I write patch modr8.module < delete-access--modr8-2.patch and I keep failing for both patches.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

Seems like that delete access patch only applies for 5.x, not HEAD (not sure why) I'm posting a re-rolled one for HEAD (i.e. 6.x)

storbaek’s picture

storbaek CreditAttribution: storbaek commented

I tested the patches on 5.x-2.3 from the 16th of August 07, including the delete access patch, and it fails on both. Perhaps I'm doing something wrong when applying the patch. If not possible to patch, I'll have to wait for a development version with the 2 patches included.

storbaek’s picture

storbaek CreditAttribution: storbaek commented

Dear Peter

Thank you very much for the programming you done to the module and the last patches. I succeded getting it to work. Apparently, I was only missing to put all files into the patching folder and not only the module file. I guess that could have been read by my statement in one of the last replies ;-)

Both patches work great. I do have one comment to when publishing nodes, which are non-published. It stills says "The post has been submitted for moderation and won't be listed publicly until it has been approved", even thought its not for moderation. It's only for moderation when its been listed publicly.

An idea could be to write that "The post has been saved and you can enter the post by going to www.website.com/subsite/edit. When you choose to publish the post for moderation, it will be listed publicly when approved". N.B.: 'Website.com/subsite' is the direct refferal to the website.

/ Dan

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Priority: Critical » Normal

This last concern seems too specific to your workflow. Probably you could tweak the theme function? http://drupal.org/node/120988

storbaek’s picture

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

Is the patch "hide-unpublished-230270-2-5x.patch" to be commited to the next version? I had to re-patch the new version to get it to work

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Reviewed & tested by the community » Needs work

That patch may not apply cleanly any longer.