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

Fix Mage_Adminhtml_Block_Template::isOutputEnabled() for invalid module #4320

Merged
merged 6 commits into from
Oct 30, 2024

Conversation

sreichel
Copy link
Contributor

Description (*)

Mage_Adminhtml_Block_Template::isOutputEnabled() returns false now for non-existing module.

Manual testing scenarios (*)

$test = new Mage_Adminhtml_Block_Template();
var_dump($test->isOutputEnabled('Mage_Catalog'));
var_dump($test->isOutputEnabled('XXX')); # returns true befor PR

@github-actions github-actions bot added Component: Core Relates to Mage_Core Component: Adminhtml Relates to Mage_Adminhtml labels Oct 29, 2024
addison74
addison74 previously approved these changes Oct 29, 2024
@sreichel sreichel marked this pull request as draft October 29, 2024 17:06
Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>
@sreichel sreichel marked this pull request as ready for review October 29, 2024 17:10
@sreichel sreichel mentioned this pull request Oct 29, 2024
@kiatng kiatng merged commit 9fc0d23 into OpenMage:main Oct 30, 2024
18 checks passed
@sreichel sreichel deleted the fix-module-ouput branch October 30, 2024 01:37
fballiano added a commit to MahoCommerce/maho that referenced this pull request Oct 30, 2024
github-actions bot pushed a commit to tmewes/maho that referenced this pull request Oct 31, 2024
…dule (#4320)

* fix isOutputEnabled

* Update tests/unit/Mage/Adminhtml/Block/TemplateTest.php

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>

* fix

OpenMage/magento-lts#4320 (comment)

* updated tests

---------

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>
github-actions bot pushed a commit to 5mehulhelp5/maho that referenced this pull request Oct 31, 2024
…dule (#4320)

* fix isOutputEnabled

* Update tests/unit/Mage/Adminhtml/Block/TemplateTest.php

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>

* fix

OpenMage/magento-lts#4320 (comment)

* updated tests

---------

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>
github-actions bot pushed a commit to mdsxbm/maho that referenced this pull request Oct 31, 2024
…dule (#4320)

* fix isOutputEnabled

* Update tests/unit/Mage/Adminhtml/Block/TemplateTest.php

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>

* fix

OpenMage/magento-lts#4320 (comment)

* updated tests

---------

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>
github-actions bot pushed a commit to MahoCommerce/maho that referenced this pull request Oct 31, 2024
…dule (#4320)

* fix isOutputEnabled

* Update tests/unit/Mage/Adminhtml/Block/TemplateTest.php

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>

* fix

OpenMage/magento-lts#4320 (comment)

* updated tests

---------

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>
github-actions bot pushed a commit to justinbeaty/maho that referenced this pull request Oct 31, 2024
…dule (#4320)

* fix isOutputEnabled

* Update tests/unit/Mage/Adminhtml/Block/TemplateTest.php

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>

* fix

OpenMage/magento-lts#4320 (comment)

* updated tests

---------

Co-authored-by: Justin Beaty <51970393+justinbeaty@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Adminhtml Relates to Mage_Adminhtml Component: Core Relates to Mage_Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants