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

Fixed bug, incorrect auto-generation Category URL for some groups of symbols (idish, cirrilic, é, ä, and other). #491

Closed
wants to merge 1 commit into from

Conversation

SKovbel
Copy link

@SKovbel SKovbel commented Feb 4, 2014

Solution:
URL formatting for Product works fine. Using Product's function formatUrlKey as basement for correct Category formatting.

Samples:
product : абвгдеёжъ -> abvgdejozh
category: абвгдеёжъ -> d-d-d-d-d-d-n-d-ns

What was done:

  • Fixed bug.
  • Created Helper for formating string to URL (Magento\Url\Helper\Format).
  • Changed function formatUrlKey in Product and Category Model to new Helper using.
  • Added function formatUrlKey for auto generation URL in Magento\Cms-Page.

*Created Helper for formating string in url (Magento\Url\Helper\Format) 
*Changed function formatUrlKey in Product and Category Model to new
Helper using.
*Added function formatUrlKey for Mage_Cms Page.
@verklov verklov self-assigned this Feb 14, 2014
@verklov
Copy link
Contributor

verklov commented Feb 14, 2014

@Mackovey , thank you very much for your contribution! We will analyze the code and post the update once the analysis is complete.

magento-team added a commit that referenced this pull request Mar 18, 2014
* Themes update:
  * LESS styles library added in pub/lib/css/
  * A new Blank theme set as default
* GitHub requests:
  * [#491](#491) -- Fixed bug, incorrect auto-generation Category URL for some groups of symbols (idish, cirrilic, , , and other).
  * [#480](#480) -- Fixing a bug for loading config from local.xml
  * [#472](#472) -- Params passed in pub/index.php being overwritten
  * [#461](#461) -- Use translates for Quote\Address\Total\Shipping
  * [#235](#235) -- Translation escaping
  * [#463](#463) -- allow _resolveArguments to do sequential lookups
  * [#499](#499) Deleted unclosed comment in calendar.css
* Fixed bugs:
  * Fixed a fatal error that occurred with a dependency in pub/errors/report.php
  * Fixed an issue where code coverage failed for Magento\SalesRule\Model\Rule\Action\Discount\CartFixedTest
  * Fixed an issue where PayPal Express Checkout redirected to the PayPal site even though the Allow Guest Checkout option was set to 'No'
  * Fixed an issue where invalid password reset link was sent when resetting customer password from the backend
  * Fixed an issue where it was not possible to download a previously created backup
  * Fixed a security issue with possibility of a XSS injection in the Integration re-authorization flow
  * Fixed an issue where Billing Agreement cancellation from the backend did not work
  * Fixed an issue with the debug section in the developer settings
  * Fixed the unreliable implementation of the fetching authorization header via SOAP
  * Fixed issues with WSDL generation error reporting
  * Fixed an issue with incorrect order of the Recurring Profile tab in Account Customer on the frontend
  * Fixed an issue when the information about a custom option of the 'File' type was not displayed correctly on the recurring profile page
  * Fixed an issue with editing Product template
  * Fixed an issue with duplicated shipping method options during checkout
  * Fixed an issue where flat indexers were re-indexed in shell when they were disabled
  * Fixed an issue where adding a wrong/nonexistent SKU using 'Order by SKU' from My Account caused a fatal error
  * Fixed an issue with the JS/CSS merging functionality
  * Fixed an issue with static view files publication tool used for the 'production' mode
* Modularity improvements:
  * Removed the deprecated GoogleCheckout functionality
  * Removed all dependencies on the RecurringPayment module
  * Removed the Sales module dependencies on Customer models/blocks
  * Renamed the RecurringProfile module to RecurringPayment
  * Resolved dependencies between the Email Templates functionality and other modules
  * Moved Core module lib-only depended components to library
  * Moved CSS URL resolving logic from publisher to the separate CSS pre-processor
  * Re-factored the View publisher
* Framework improvements:
  * Added restrictions on the data populated to the Service Data Object
  * Renamed Data Transfer Object to Service Data Object
  * Updated the view files population tool to support LESS
* Customer Service usage:
  * Refactored the Tax module to use Customer service layer
  * Refactored Customer module Adminhtml internal controllers and helper to use Customer services
  * Added and updated the Customer service APIs
  * Exposed Customer services as REST APIs
* Indexer implementation:
  * Implemented a new optimized Product Price Indexer
* Updated various PHPDoc with the parameter and return types
@verklov
Copy link
Contributor

verklov commented Mar 19, 2014

@Mackovey, the team has processed your pull request. The fix is available in version dev69.

@verklov verklov closed this Mar 19, 2014
vpelipenko added a commit that referenced this pull request Jul 24, 2015
MAGETWO-40680: Update README.md with new content
magento-engcom-team added a commit that referenced this pull request Mar 30, 2019
…491

 - Merge Pull Request magento/graphql-ce#491 from 06romix/graphql-ce:issue-478
 - Merged commits:
   1. 4f19d03
   2. 43f5bce
   3. 4bdc30e
   4. f9ecace
   5. 60d40e1
   6. f01dc00
   7. 16ce046
   8. c2f15b3
   9. 14d5c3d
   10. 2643424
   11. cf4eb72
   12. 829a39b
   13. d0f3920
   14. 2ceceb7
   15. 84a21a4
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.

2 participants