Problem/Motivation

We need to create a hook update to update all modals for published = TRUE

Proposed resolution

Create this

CommentFileSizeAuthor
#2 modal_page-3098094.patch2.54 KBrenatog
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RenatoG created an issue. See original summary.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Active » Needs work
FileSize
modal_page-3098094.patch2.54 KB

The first patch, but not work yet

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented

Hello @renatoG,
This hook below is already updating all modal to published = TRUE. Do you agree with me? Can we close this issue?

/**
 * Implements hook_update_N().
 *
 * Hook to update the entity modal_page_modal.
 */
modal_page_update_8202(){
  create_storage_published();
  populate_published_field();

  \Drupal::service('cache.render')->invalidateAll(); 
}
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented

Yeah, Paulo. We did it in another issue and we forget this issue opened.

So yeah, we can close that.

Good catch! Thank you!

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented
Status: Needs work » Fixed

Fixed because it is already done.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Fixed » Closed (fixed)