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

Wrong parameter for getting base url for 'media' path in "Image" form element. #1025

Merged
merged 1 commit into from
Feb 13, 2015
Merged

Wrong parameter for getting base url for 'media' path in "Image" form element. #1025

merged 1 commit into from
Feb 13, 2015

Conversation

kl177
Copy link
Contributor

@kl177 kl177 commented Feb 6, 2015

Form element "image" doesn't load 'media' path.
http://dl1.joxi.net/drive/0000/0178/12466/150206/57a3f0519b.jpg

\Magento\Framework\UrlInterface::getBaseUrl accepts an array instead of a string.

@otoolec
Copy link
Contributor

otoolec commented Feb 7, 2015

Thanks for submitting this pull request. We will process it internally through ticket MAGETWO-33727 and merge your pull request after that.

@otoolec otoolec self-assigned this Feb 10, 2015
@vpelipenko
Copy link
Contributor

CR: passed
Builds: green
Resolution: OK to merge

vpelipenko added a commit that referenced this pull request Feb 13, 2015
Wrong parameter for getting base url for 'media' path in "Image" form element.
@vpelipenko vpelipenko merged commit a0727fb into magento:develop Feb 13, 2015
otoolec added a commit that referenced this pull request Feb 13, 2015
…in "Image" form element. #1025

- Import UrlInterface to short line to under 120 characters
- Add integration test to catch regression
- Modify unit test to catch regression
magento-team added a commit to abeeskau/magento2-community-edition that referenced this pull request Feb 20, 2015
* Various improvements:
    * Existing Builders were replaced with DataFactories in Customer and Tax modules
    * Refactored controller actions in the Checkout and CMS modules
    * Increased coverage with static tests for `.phtml` files
    * Moved Cookie related functionality from `Theme` and `Core` modules into a new `Cookie` module
    * Moved minfication configuration settings to the `View` library level
* UI improvements:
    * Restyled installation wizard
    * Prepared styles for Dashboard in the Backend area
* Framework improvements:
    * Added `setCustomAttribute` and `setCustomAttributes` methods to `ExtensibleDataInterface`
    * Added setter methods to data object interfaces
    * Replaced `Builders` with `Factories`
    * Added `DataObjectHelper.php` which contains the common set of methods of all builders
    * Refactored `__()` to return `Phrase` object
    * Allowed usage of `LocalizedException` on the framework's library level
    * Added expiration/lifetime management of frontend resources
    * Unified MTF configurations format for Framework, TestCase variations and TestCase scenario configurations
* Fixed bugs:
    * Fixed an issue with product reviews list paging
    * Fixed an issue where sold products were not displayed in Bestsellers
    * Fixed an issue with image rendering on the CMS page on Frontend when `webserver rewrites = no`
* GitHub requests:
    * [#790](magento/magento2#790) -- Magento API fails in a CGI env (zf1 issue)
    * [#909](magento/magento2#909) -- Manage Titles in popup window front-end issue
    * [#996](magento/magento2#996) -- Pager block should support url "fragment".
    * [#985](magento/magento2#985) -- Allow camelcase in vendorname for menus
    * [#1025](magento/magento2#1025) -- Wrong parameter for getting base url for 'media' path in "Image" form element.
magento-team pushed a commit that referenced this pull request Apr 14, 2017
[Folks] Api Annotation Coverage of OMS-related Modules
VitaliyBoyko pushed a commit to VitaliyBoyko/magento2 that referenced this pull request Jun 22, 2018
MSI-724: Implement IsCorrectQtyConditionTest::testExecuteWithQtyIncrements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants