After #2936263: User's temp stores has been moved to core for 8.5.x, the "user.shared_tempstore" is still in use in some parts, mainly in the Form classes. They should be replaced by the new "tempstore.shared" service name.

CommentFileSizeAuthor
#2 ctools-user_temp_store_in_forms-3033634-2.patch5.96 KBmanuel.adan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

manuel.adan created an issue. See original summary.

manuel.adan’s picture

manuel.adan
Location 🌌
CreditAttribution: manuel.adan at Arcovía Digital for Asociación TecnoAriadna commented
Assigned: manuel.adan » Unassigned
Status: Active » Needs review
FileSize
ctools-user_temp_store_in_forms-3033634-2.patch5.96 KB

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented
Status: Needs review » Fixed

Thanks, sorry for missing that on the previous commit. I usually double check those kinds of things on commit.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid as a volunteer commented

Can we possibly get an updated 3.2 release? Anything that uses the wizards is now broken because not the same tempstore services were used in 3.1 but were fixed in this commit.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented

@Dave Reid I will shortly, waiting on one other bug fix and this security issue to wrap up.

Status: Fixed » Closed (fixed)

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