Feature is overridden always

It looks like this was discussed when exportables support was added: #825088-51: Exportables support for CAPTCHA points, though it seems like it was not dealt with there.

I think that removing hook_captcha_default_points_alter() would prevent several defaults from getting set, so I'm not sure if that alone would be an acceptable fix to the issue.

The code removed in this patch is basically providing some default captchas. That makes sense now that captcha data are exportable. The problem seems to be that, in introducing a new form of providing default captchas, we didn't remove the old one.

Should we at the same time replace these defaults? If so, there would be two options:

• Replace them with default captchas.
• Replace them by creating them through a hook_install() implementation.

Of the two, I'd say the second is probably cleaner. It initializes the captchas, but doesn't interfere if for example another module wants to provide exported versions of the same captchas.

The problem seems to be that, in introducing a new form of providing default captchas, we didn't remove the old one.

Hummm, I'm probably missing something about the history of the exportables feature addition. What is the "old" way that was not removed?

Replace [defaults] by creating them through a hook_install() implementation.

You noted that this was probably the preferred way to go. That makes sense as it's much cleaner, but I think it would also remove the ability for the module to easily add defaults for new forms that are added after module installation (e.g., a new content type is added). By using hook_captcha_default_points_alter() it seems like the defaults are always re-evaluated, and missing forms not already captured in config are detected, whenever ctools_export_load_object() is called. This is of course very handy when the admin form is loaded (new defaults can sanely be suggested real-time) but it's not so great for features integration.

I personally don't have a lot of programmatic experience with ctools exportables, so I'm not sure if I know what the best practice would be for these cases... additional default detection logic in captcha_admin_settings()?

I am getting the same issue; features that have Captcha points are showing as overridden. When I attempt to recreate the feature, the Captcha configuration/point isn't shown in the Components section. I am unable to remove the Captcha point from the feature in the UI, so it permanently shows overridden.

Revert or recreation does not solve the problem.

Same here

Having this same issue and the patch doesn't help.

This patch seems to have fixed the issue for me.

The patch worked for me thanks.

As per #5 there should be defaults for new forms.
A way has to be found to to work together with features.

Drupal 7 is no longer supported, closing.