Every time I enable or disable a module, I get this message:

Conditional Fields settings have been updated.

It doesn't matter what module I am enabling... and I have gotten this message on content type manage fields pages as well.

Any ideas?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Rosamunda’s picture

Title: Get message every time I visit my modules page » Message "Conditional Fields settings have been updated" shows everytime this module is called

I have this message even when I create new nodes that have conditional fields...

upupax’s picture

Same issue..

peterpoe’s picture

Component: User interface » Code

I can't reproduce this issue.
That message should appear only when a content type is updated, and its machine-readable name changed. It's called by this module's implementation of hook_node_type.
The only reason that I can think why this should happen is that some module changes the name of a node type with conditional fields in it when you enable/disable it.
Could you please paste here a list of modules/versions enabled in your site?

upupax’s picture

admin_menu
admin_menu_dropdown
apply_for_role
auto_nodetitle
backup_migrate
ca
conditional_fields
content
content_profile
content_profile_registration
content_taxonomy
content_taxonomy_options
devel
dhtml_menu
download_count
fieldgroup
filefield
filefield_token
hierarchical_select
hs_taxonomy
imageapi
imageapi_gd
imagefield
jquery_ui
jquery_update
nodereference
number
optionwidgets
rules
taxonomy_menu
test_gateway
text
token
token_actions
transliteration
trigger
uc_ajax_cart
uc_cart
uc_credit
uc_order
uc_payment
uc_product
uc_store
uc_vat_number
views
views_embed_form
views_ui

upupax’s picture

Sorry, I forgot to put the versions: they are all the last available recommended version for drupal 6.x.
For Token and for those that still don't have a stable version (like hierarchical_select or admin_menu_dropdown) I installed the -dev version.

peterpoe’s picture

Found the problem. Seems that hook_node_type is called on the modules page from book, poll and forum modules, with no 'old_type' variable. Will provide a patch shortly.

peterpoe’s picture

Status: Active » Needs review
FileSize
1.01 KB

Here's the patch. Added also a check to avoid unnecessary db queries.

peterpoe’s picture

New patch. Variables might have to change anyway, so I removed the check for data. Removed the message, which can be confusing.

upupax’s picture

Seems to work for me..

peterpoe’s picture

Status: Needs review » Reviewed & tested by the community

Fine! Committing to dev.

sbsimmons86’s picture

Thanks for the patch... worked great.

peterpoe’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.