Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closes #2081: Upgrade Drupal core to 10.0.9 #2354

Merged
merged 47 commits into from
Jun 17, 2023
Merged

Closes #2081: Upgrade Drupal core to 10.0.9 #2354

merged 47 commits into from
Jun 17, 2023

Conversation

joeparsons
Copy link
Member

@joeparsons joeparsons commented Apr 19, 2023

Description

  • Upgrades Drupal core to 10.0.9
  • Adds CKEditor 4 contrib module
  • Upgrades various contrib modules to D10-compatible versions (including config_sync/config_distro suite of modules)
  • Updates core and contrib patches
  • Fixes various issues with tests
    • Replaces usage of seven theme in tests with claro since seven is no longer available in D10
    • Adds new patch for Bootstrap Barrio JS library issue that was causing an error that broke a PHPUnit test
    • Updates usage of str_replace migrate process plugin provided by migrate_plus in various migrations to fix failing migration tests
  • Fixes various issues reported by phpstan and PHPCS
  • Fixes an issue that @tadean noticed involving once.js that affected the Event Calendar and Publication date picker
  • Updates lando and ProboCI configs
  • Removes lando/ddev tooling for Drupal console
  • Updates phpstan and PHPCS configs
  • Replaces drupal/background_image_formatter and drupal/smart_title with az-digital D10-compatible forks

Related PRs

These will need to be merged in conjunction with this one:

Some of the changes included here are also available in individual PRs where they may be easier to review/test/merge in isolation (and probably should be):

Related issues

Closes #2081

How to test

  • General testing
  • CKeditor testing

Types of changes

Arizona Quickstart (install profile, custom modules, custom theme)

  • Patch release changes
    • Bug fix
    • Accessibility, performance, or security improvement
    • Critical institutional link or brand change
    • Adding experimental module
    • Update experimental module
  • Minor release changes
    • New feature
    • Breaking or visual change to existing behavior
    • Upgrade experimental module to stable
    • Enable existing module by default or database update
    • Non-critical brand change
    • New internal API or API improvement with backwards compatibility
    • Risky or disruptive cleanup to comply with coding standards
    • High-risk or disruptive change (requires upgrade path, risks regression, etc.)
  • Other or unknown
    • Other or unknown

Drupal core

  • Patch release changes
    • Security update
    • Patch level release (non-security bug-fix release)
    • Patch removal that's no longer necessary
  • Minor release changes
    • Major or minor level update
  • Other or unknown
    • Other or unknown

Drupal contrib projects

  • Patch release changes
    • Security update
    • Patch or minor level update
    • Add new module
    • Patch removal that's no longer necessary
  • Minor release changes
    • Major level update
  • Other or unknown
    • Other or unknown

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@joeparsons joeparsons self-assigned this Apr 19, 2023
@joeparsons joeparsons added 2.7.x only dependencies Pull requests that update a dependency file Drupal Waiting on a fix from the Drupal community high priority Must get done for this milestone labels Apr 19, 2023
@tadean
Copy link
Contributor

tadean commented Jun 15, 2023

It seems like Storage comparer issue in the removed config_distro patches might still exist (though the patch no longer applies), but I'm not sure...this was for #428 I think . Here is a potentially rerolled patch, though maybe the actual cause could be elsewhere...

@camikazegreen
Copy link
Contributor

camikazegreen commented Jun 16, 2023

When attempting to edit this News article as an Admin or Content Admin, I get this error:
Screenshot 2023-06-16 at 9 29 49 AM

I have verified that this occurs with any News that I have tried to edit, but does not happen for Pages, Events or People.

@joeparsons
Copy link
Member Author

It seems like Storage comparer issue in the removed config_distro patches might still exist (though the patch no longer applies), but I'm not sure...this was for #428 I think . Here is a potentially rerolled patch, though maybe the actual cause could be elsewhere...

Thanks @tadean I've restored the (re-rolled) patch.

@Carris-Toupin
Copy link
Contributor

Carris-Toupin commented Jun 16, 2023

I checked the CK editor functions as azadmin in a Page and everything works as it should, especially the embed features. Let me know if you want me to test as a specific user or in a specific Content type.

@Carris-Toupin Carris-Toupin self-requested a review June 16, 2023 18:22
Carris-Toupin
Carris-Toupin previously approved these changes Jun 16, 2023
tadean
tadean previously approved these changes Jun 16, 2023
@tadean
Copy link
Contributor

tadean commented Jun 16, 2023

After taking a D9 build and moving it to #2354, a cache clear was required to get the site working again. The following database updates were available:

 ------------------------ ----------- --------------- ---------------------------------------------------------------- 
  Module                   Update ID   Type            Description                                                     
 ------------------------ ----------- --------------- ---------------------------------------------------------------- 
  better_exposed_filters   8003        hook_update_n   8003 - Provide upgrade path from 8.x-4.x to 8.x-5.x.            
  user                     10000       hook_update_n   10000 - Remove non-existent permissions created by migrations.  
 ------------------------ ----------- --------------- ---------------------------------------------------------------- 

The following config_distro changes were present for import:

 ---------------- ---------------- ------------------------------------------------ 
 Extension        Operation type   Label                                           
---------------- ---------------- ------------------------------------------------ 
 az_core          create           Enterprise Thumbnail                            
 az_core          create           Enterprise Thumbnail Small                      
 az_core          create           taxonomy_term.field_az_attribute_type           
 az_core          create           taxonomy_term.field_az_attribute_key            
 az_core          create           node.field_az_enterprise_attributes             
 az_core          create           Enterprise Attributes                           
 az_core          create           Attribute Type                                  
 az_core          create           Attribute Key                                   
 az_core          create           taxonomy_term.az_enterprise_attributes.default  
 az_core          create           taxonomy_term.az_enterprise_attributes.default  
 az_core          create           Taxonomy List Pages                             
 az_core          update           az_core.settings                                
 az_core          update           smart_title.settings                            
 az_media         update           Standard                                        
 az_news          create           Enterprise Attributes                           
 az_news          create           node.az_news.az_row                             
 az_news          update           node.az_news.default                            
 az_news          update           node.az_news.teaser                             
 az_news          update           node.az_news.default                            
 az_news          update           node.az_news.az_minimal_media_list              
 az_news          update           node.az_news.az_medium_media_list               
 az_news          update           node.az_news.az_media_list                      
 az_news          update           node.az_news.az_marquee                         
 az_news          update           node.az_news.az_feature                         
 az_news          update           node.az_news.az_card                            
 az_person        update           node.az_person.default                          
 az_publication   create           node.az_publication.az_card                     
 az_publication   create           AZ Publications Curate                          
 az_publication   update           node.az_publication.default                     
 az_publication   update           AZ Publications                                 
 node             update           Content                                         
 user             update           Anonymous user                                  
 user             update           Authenticated user                              
 az_quickstart    update           Content editor                                  
 az_quickstart    update           Content administrator                           
---------------- ---------------- ------------------------------------------------ 

(This existing site had az_publication already enabled.)

The ckeditor CKEditor 4 (contrib) (Deprecated) module showed up as enabled, likely because it has the same identifier as the old core equivalent that was in use prior to D10.

CK Editor still seemed to work for me, and had styles and embeds available.

ck4

@camikazegreen
Copy link
Contributor

Confirming that the latest version has resolved the issue with editing News.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.7.x only dependencies Pull requests that update a dependency file Drupal Waiting on a fix from the Drupal community high priority Must get done for this milestone needs-CWS-testing Needs manual pre/post release testing by Campus Web Services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drupal 10 compatibility
5 participants