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

CommentFileSizeAuthor
#5 journal.test3.58 KBevoltech

Comments

sun’s picture

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.

marcp’s picture

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

sun’s picture

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.

marcp’s picture

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.

evoltech’s picture

StatusFileSize
new3.58 KB

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?

evoltech’s picture

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)

sun’s picture

Status: Patch (to be ported) » Closed (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.