-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Issue 12506: Fixup typo getDispretionPath -> getDispersionPath #12507
Conversation
Not sure if all pull requests need an issue, so I created one just to be sure. |
@@ -611,7 +611,7 @@ public static function getNewFileName($destinationFile) | |||
* @param string $fileName | |||
* @return string | |||
*/ | |||
public static function getDispretionPath($fileName) | |||
public static function getDispersionPath($fileName) |
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.
Renaming public methods is backward incompatible.
Please, refactor this codes with following steps:
- introduce new method getDispersionPath
- move logic from old getDispretionPath method to created getDispersionPath
- mark old method as @deprecated
- make proxy call from old method to new
public static function getDispretionPath($fileName)
{
return self::getDispersionPath($fileName);
}
Backward compatible development rules described in http://devdocs.magento.com/guides/v2.2/contributor-guide/backward-compatible-development/
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.
You're right, I was too quick. Made the necessary changes
[EngCom] Public Pull Requests - 2.2-develop - MAGETWO-84981: Trying to get data from non existent products #12539 - MAGETWO-84979: [Backport 2.2-develop] Fix swagger-ui on instances of Magento running on a non-standard port #12541 - MAGETWO-84903: Added namespace to product videos fotorama events #12469 - MAGETWO-84862: [Backport 2.2-develop] #11409: Too many password reset requests even when disabled in settings #11435 - MAGETWO-84856: Issue 12506: Fixup typo getDispretionPath -> getDispersionPath #12507 - MAGETWO-84808: 12110: Missing cascade into attribute set deletion. #12167 - MAGETWO-83503: [2.2] - Add command to view mview state and queue #12122 - MAGETWO-80223: Fix syntax of expectException() calls #11099
Description
Fixup typo getDispretionPath -> getDispersionPath
Fixed Issues (if relevant)
Contribution checklist