First shot.

See http://drupal.org/project/form_controller for further information.

CommentFileSizeAuthor
#6 journal.formids.patch2.7 KBsun
#5 journal-DRUPAL-5.form_controller.patch2.04 KBsun
#4 journal-DRUPAL-5.formids-variable.patch2.93 KBsun
#1 journal-DRUPAL-5.form_controller.patch4.15 KBsun
journal-DRUPAL-5.form_controller.patch4.42 KBsun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Active » Needs review
StatusFileSize
new journal-DRUPAL-5.form_controller.patch4.15 KB

Re-rolled.

pcoughlin’s picture

pcoughlin commented

I have installed this patch ... now what?
What do I need to check for?
What should have changed?
I still have multiple Journal entries on pages (for example the Events)

Are there any instructions on how to remove Journal entries on selected pages?

Also, it would be nice to only have the journal appear for certain roles.

Thanks ... Paul

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

This patch integrates Journal with http://drupal.org/project/form_controller
You need to have Form controller installed + enabled to actually have any use of this patch. If both modules are enabled, and your user is assigned to a role that permits usage of both modules, then you can enable, disable, or even require a journal entry for individual forms.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new journal-DRUPAL-5.formids-variable.patch2.93 KB

Comitted attached patch, which allows to alter the pre-defined array of form ids via settings.php.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new journal-DRUPAL-5.form_controller.patch2.04 KB

Re-rolled hook_form_controller() patch.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new journal.formids.patch2.7 KB

I've additionally committed attached patch that adds form ids for Views 2 (de-facto core) and explains how to override the variable in settings.php.

Aside from that, patch in #5 is still the topic of this issue. ;)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Closed (won't fix)
sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Closed (won't fix) » Needs review

Re-opening.

bwinett’s picture

bwinett commented

I submitted a patch to make this a configuration option in v6.

pcoughlin’s picture

pcoughlin commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

The last comment was three years ago and 6 years ago before that!
Also, this is for Drupal 5.x.
I am closing and if anyone is still interested, then please re-open.

Thanks.