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

Removed deprecated method Mage::setIsDownloader() #2379

Merged
merged 1 commit into from
Aug 10, 2022
Merged

Removed deprecated method Mage::setIsDownloader() #2379

merged 1 commit into from
Aug 10, 2022

Conversation

fballiano
Copy link
Contributor

Mage:: setIsDownloader() is deprecated and unused in the whole code and it can be safely removed. This PR targets v20.

@github-actions github-actions bot added the Mage.php Relates to app/Mage.php label Aug 8, 2022
@sreichel
Copy link
Contributor

sreichel commented Aug 9, 2022

Remove it from 1.9 too?

@fballiano
Copy link
Contributor Author

I don't know, I felt I wasn't supposed to remove stuff from the LTS. although this method is totally useless...

@fballiano fballiano merged commit b1098b2 into OpenMage:20.0 Aug 10, 2022
@fballiano fballiano deleted the remove_set_is_downloader branch August 10, 2022 18:10
@github-actions
Copy link
Contributor

Unit Test Results

1 files  ±0  1 suites  ±0   0s ⏱️ ±0s
0 tests ±0  0 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
7 runs  ±0  7 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b1098b2. ± Comparison against base commit 5cca7d5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mage.php Relates to app/Mage.php
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants