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

Error in the "_IsValidSource" method in Varien/Io/File.php #1178

Closed
felipegomesdev opened this issue Sep 2, 2020 · 0 comments · Fixed by #1302
Closed

Error in the "_IsValidSource" method in Varien/Io/File.php #1178

felipegomesdev opened this issue Sep 2, 2020 · 0 comments · Fixed by #1302
Labels

Comments

@felipegomesdev
Copy link

Hello.

After this change is part of the code (#966)

The SOAP API Feature of creating a product image has stopped working. Look documentation

Well, the whole image has a null value. For example:

$src = file_get_contents("https://pbs.twimg.com/media/EelBMRwXsAEcrM9.png");

if (strpos($src, chr(0)) === false){
    echo "success...";
}else {
    echo "error...";
}

Therefore, it is not possible to pass the _IsValidSource method in Varien/Io/File.php

Same problem at #1125

digitalpianism added a commit to digitalpianism/magento-lts that referenced this issue Nov 10, 2020
sreichel added a commit that referenced this issue Mar 11, 2021
#1302)

* bump version

* Fix _addUrlRewrite() ignoring collection store scope. (#510)

While Mage_Catalog_Model_Resource_Product_Collection::addUrlRewrite()
does respect any set storeId on the collection it's helper that does
the actual work is hard-coded to the current active store.

* Add doc comments to image related classes (#1146)

* Prevent duplicate entry when updating salesrule_coupon_usage (#1117)

* remove $timesUsed > 0 check to prevent duplicate entry

* Prevent $timesUsed from going less than 0

* Fix a bug where media upload via API are not possible anymore: #1178

* revert unwanted changes

* Update lib/Varien/Io/File.php

Co-authored-by: Flyingmana <flyingmana@googlemail.com>
Co-authored-by: Colin Mollenhour <colin@mollenhour.com>
Co-authored-by: Erik Dannenberg <ed@reshape.de>
Co-authored-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Co-authored-by: Dean Williams <me@deanwilliams.org>
Co-authored-by: sv3n <github-sr@hotmail.com>
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 a pull request may close this issue.

1 participant