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

fix(SHS-6113): Exception on duplicate nodes being cleaned up #1770

Conversation

joegl
Copy link
Contributor

@joegl joegl commented Mar 4, 2025

Summary

Fix for the duplicate node database update failure.
$permissions is sometimes not an array and fails when being ran through array_unique() (this is possibly due to the per_node setting being enabled).

Urgency

High

@ahughes3 ahughes3 temporarily deployed to Tugboat March 4, 2025 17:23 Destroyed
@ahughes3 ahughes3 temporarily deployed to Tugboat March 4, 2025 17:26 Destroyed
@joegl joegl changed the title fix(SHS-6113): exception on some nodes being cleaned up fix(SHS-6113): Exception on some nodes being cleaned up Mar 4, 2025
@joegl joegl changed the title fix(SHS-6113): Exception on some nodes being cleaned up fix(SHS-6113): Exception on duplicate nodes being cleaned up Mar 4, 2025
@joegl
Copy link
Contributor Author

joegl commented Mar 4, 2025

I ran this on the Economics site locally and it worked:

>  [notice] Update started: su_humsci_profile_update_9724
>  [notice] Update completed: su_humsci_profile_update_9724
>  [notice] Update started: su_humsci_profile_update_9725
>  [notice] Update completed: su_humsci_profile_update_9725
>  [notice] Update started: su_humsci_profile_update_9726
>  [notice] Update completed: su_humsci_profile_update_9726

I'm going to merge this one.

@joegl joegl merged commit c7961ad into 11.8.1-release Mar 4, 2025
22 of 57 checks passed
@joegl joegl deleted the SHS-6113_Temp-UnavailableError-issue-with-Private-Pages-and-Menus branch March 4, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants