The current tests are enough to show this you just need to rewrite module_implements :D actions_synchronize badly needs this TRUE. 6.x too. Simply, when you resync you need to reset as well.

CommentFileSizeAuthor
resync_reset.patch513 byteschx

Comments

dave reid’s picture

Tested and works fine.

Looking around the actions_synchronize code, I'm not sure even why it needs the $actions_in_code parameter. The only time it isn't given actions_list() for that is when it is called from system_modules_submit. I'll file a separate issue for it, but just wanted to know what you think chx.

dave reid’s picture

See #306540: Orphaned assigned actions still triggered and cannot be removed for proposed change to actions_synchronize.

chx’s picture

There is one thing for sure, #298600: DBTNG: make module_implements work regardless of bootstrap phase is horribly broken without this.

dave reid’s picture

Sorry about getting off track. I agree with chx that this is needed and the patch is good and tested fine. RTBC

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

damien tournoud’s picture

Version: 7.x-dev » 6.x-dev
Status: Closed (fixed) » Reviewed & tested by the community

Need to get into 6.x too.

gábor hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Committed to Drupal 6.x, thanks.

Status: Fixed » Closed (fixed)

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