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.

Sign up for GitHub

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

Jump to bottom

Review: 1509-unable-to-update-publisher-id #1513

Merged
merged 1 commit into from
Aug 1, 2024
Merged

Review: 1509-unable-to-update-publisher-id #1513

merged 1 commit into from
Aug 1, 2024

Conversation

PG-Momik
Copy link
Collaborator

  • Fix : Unable to update publisher

@PG-Momik PG-Momik linked an issue Jul 30, 2024 that may be closed by this pull request
Unable to update publisher ID #1509
Closed
@what-the-diff What The Diff
Copy link

what-the-diff bot commented Jul 30, 2024

PR Summary

  • Update $orgPublished variable for null check
    The change has been made in the OrganizationService.php file, specifically on line 376, to ensure the $orgPublished variable is checked for null values. If it turns out to be null, an empty array will be assigned instead. This update enhances the robustness of our code by preventing potential issues related to null values in the future.

@PG-Momik PG-Momik force-pushed the 1509-unable-to-update-publisher-id branch from 496242a to 1ece0e1 Compare July 31, 2024 11:26
@PG-Momik PG-Momik force-pushed the 1509-unable-to-update-publisher-id branch from 1ece0e1 to 663322f Compare August 1, 2024 03:50
@A4family A4family merged commit 4bad3f2 into main Aug 1, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@A4family A4family A4family approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Unable to update publisher ID
2 participants
@PG-Momik @A4family