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 500 #501

Merged
merged 1 commit into from
Nov 2, 2023
Merged

fix 500 #501

merged 1 commit into from
Nov 2, 2023

Conversation

lehaa22
Copy link
Contributor

@lehaa22 lehaa22 commented Nov 2, 2023

Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues #500

Copy link

what-the-diff bot commented Nov 2, 2023

PR Summary

  • Modification of Migration File
    The migration file M211218163000JobQueueSize.php underwent a change.

  • Changing the Parameter in up() method
    The up() function, used to apply a new change (like creating a database table or adding a new column to it), now uses $this->tableName in lieu of the earlier {{%queue}}. This makes the code more standardized and efficient as it now refers to the table name directly from the class context.

  • Altering the Parameter in down() method
    Similarly in the down() function, which is used to revert the changes made by the up() function (drop the table or the column in case of a rollback), the parameter is also changed to $this->tableName replacing {{%queue}}. This ensures consistency across the code and improves code maintainability.

@bizley bizley added the type:bug Bug label Nov 2, 2023
@bizley bizley added this to the 2.3.7 milestone Nov 2, 2023
@bizley bizley merged commit 9e9d465 into yiisoft:master Nov 2, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants