When running drush config-export, the majority of the Force Password Change settings do not get exported to config.

Did some poking around, and looks like it's because Force Password Change is storing most of the configuration in separate tables:

force_password_change_expiry
force_password_change_roles
force_password_change_uids

It seems like the force_password_change_expiry and force_password_change_roles should both be configuration, not separate tables.

CommentFileSizeAuthor
#12 force_password_change-2886166-12.patch8.87 KBYurkinPark
#10 force_password_change-2886166.patch6.98 KBvoviko

Issue fork force_password_change-2886166

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

marcaddeo created an issue. See original summary.

larisse’s picture

larisse commented

Changing to 2.0.x version.

larisse’s picture

larisse commented
Version: 8.x-1.x-dev » 2.0.x-dev
andypost’s picture

andypost
he/him
Primary language Russian
commented

Moreover roles should be added to config dependencies

voviko made their first commit to this issue’s fork.

vacho’s picture

vacho commented

All registers in form /admin/config/people/force_password_change should be registered in configuration, to let export.

Also, this can let to retrieve saved values for roles in the "Force users in the following roles to change their password" part.

larisse’s picture

larisse commented
Parent issue: » #3254813: Plan for Force Password Change

voviko opened merge request !7

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs work

The MR has many changes which are out of scope and should be removed, also looks like fork was made from wrong branch (use 2.x)

voviko’s picture

voviko commented
StatusFileSize
new force_password_change-2886166.patch6.98 KB
KondratievaS’s picture

KondratievaS commented
Status: Needs work » Reviewed & tested by the community

Tested patch from #10 - works as expected

YurkinPark’s picture

YurkinPark commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new force_password_change-2886166-12.patch8.87 KB

Reroll for latest 2.0.x branch state

  • larisse committed b9d6ada on 2.0.x authored by voviko 
    Issue #2886166 by voviko, YurkinPark, larisse, andypost, marcaddeo,...
larisse’s picture

larisse commented
Status: Needs review » Fixed

I applied and tested the reroll in #12. Thanks all for contributed for it. :)

Status: Fixed » Closed (fixed)

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

heddn closed merge request !7