-
Notifications
You must be signed in to change notification settings - Fork 891
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
Better reversible migrations #1164
Conversation
Enables otherwise irreversible migrations to become reversible, and makes it possible to use a single migration where previously two were required.
@@ -66,6 +66,11 @@ | |||
protected $input; | |||
|
|||
/** | |||
* @var bool | |||
*/ | |||
protected $goingUp; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would not be a boolean or be accurate when using the up and down methods, would it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
No, but you move the setter call up so it would. It seemed superfluous.
|
It does seem superfluous, but it is not. Having the state of an object consistent is important because you don't know the different permutations a particular object can be used |
So should I close the #788 ? |
@mvrhov yes |
Thanks! |
@lorenzo I know you already merged but? Wouldn't it be better if |
@mvrhov Sure, it like that.. care to open a new PR with the change? |
Sorry I don't have time :/ Maybe @Roy-Orbison can find some. |
Enables otherwise irreversible migrations to become reversible, and makes
it possible to use a single migration where previously two were required.
Pursuant to #1151