Implement hook_form_controller()

sun - February 29, 2008 - 03:28
Project:Journal
Version:5.x-1.x-dev
Component:Forms
Category:feature request
Priority:normal
Assigned:sun
Status:needs review
Description

First shot.

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

AttachmentSize
journal-DRUPAL-5.form_controller.patch4.42 KB

#1

sun - April 16, 2008 - 20:43
Status:active» needs review

Re-rolled.

AttachmentSize
journal-DRUPAL-5.form_controller.patch 4.15 KB

#2

pcoughlin - April 24, 2008 - 00:08

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

#3

sun - April 24, 2008 - 00:51

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.

#4

sun - August 15, 2008 - 19:35

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

AttachmentSize
journal-DRUPAL-5.formids-variable.patch 2.93 KB

#5

sun - August 15, 2008 - 19:36

Re-rolled hook_form_controller() patch.

AttachmentSize
journal-DRUPAL-5.form_controller.patch 2.04 KB

#6

sun - August 16, 2008 - 20:15

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. ;)

AttachmentSize
journal.formids.patch 2.7 KB

#7

sun - August 2, 2009 - 22:18
Status:needs review» won't fix

#8

sun - October 7, 2009 - 11:01
Status:won't fix» needs review

Re-opening.

 
 

Drupal is a registered trademark of Dries Buytaert.