Journal entry does not save at admin/content/node-type/[content-type-name]/display

bonobo - June 2, 2009 - 05:42
Project:Journal
Version:6.x-1.2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:duplicate
Description

When making adjustments to the field visibility settings (part of CCK), the journal entries saved when at admin/content/node-type/[content-type-name]/display are not saved.

Possibly, this could be related to #474422: Journal entry does not save at admin/build/block

#1

sun - June 5, 2009 - 02:51

All of those bugs are caused by additional form #submit handlers of other modules that intercept the form submission and/or remove any other form submit handlers from the queue.

#2

marcp - June 8, 2009 - 17:21

So if Journal's weight was higher than other contrib modules, then this would be a non-issue?

#3

sun - June 8, 2009 - 17:58

Most often, unfortunately not. Without looking at the issue at hand, the usual case is that another module replaces all form validate and/or submit handlers.

#4

marcp - June 8, 2009 - 20:57

I haven't tried to reproduce with CCK installed yet, but saving a journal entry at admin/content/node-type/page works just fine, so CCK must be interfering.

#5

evoltech - June 11, 2009 - 08:33

I am going to be submitting a patch to fix this tomorrow. In the meantime I have started to integrate in simpletest into the module. Included below is the journal.test file for this with a functionality test that triggers this bug. I am assuming you want patches submitted as unified diffs?

AttachmentSize
journal.test 3.58 KB

#6

evoltech - June 12, 2009 - 02:25
Status:active» patch (to be ported)

I confirmed that this is the same issue as 474422. I have submitted a patch, with updated journal.test to test this specific issue as well as 474422 (http://drupal.org/node/474422)

#7

sun - June 12, 2009 - 02:58
Status:patch (to be ported)» duplicate

Marking as duplicate of #474422: Journal entry does not save at admin/build/block. You can follow up on that issue to track its status instead. If any information from this issue is missing in the other issue, please make sure you provide it over there.

However, thanks for taking the time to report this issue.

 
 

Drupal is a registered trademark of Dries Buytaert.