-
-
Notifications
You must be signed in to change notification settings - Fork 436
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
don't redirect to "downloader" (which doesn't exists) #1523
Conversation
Downloader was removed with #952 so it's useless to redirect to the downloader subdir. Also, I think the purpose of checkin if the MAGENTO_ROOT . '/app/Mage.php' is useless, one file check for every request while magento wouldn't work at all if it didn't exist, I think that check has to be removed, there's no point in executing it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree that the check is not useful. For new install, it may be useful to inform that file is missing, but PHP would throw an error anyway. For development or production, it's totally useless.
I think this should really be merged, the downloader doesn't exist since some time and this check is just a waste of CPU clocks, let's close this and move on to other stuff no? ;-) I know I'm a pain in the *** but I keep checking for my issues/PR and so I don't use the time to do other stuff for the project. |
what should we do about this? |
@Flyingmana @sreichel @kiatng I think this one could be easily merged, otherwise close it and let's forget it (but I can't see why) |
I agree that this should be merged to v20. |
In the main index.php there's a check for the app/Mage.php (which is 99.9% useless) and a redirect to the "magento downloader".
Downloader was removed with #952 so it's useless to redirect to the downloader subdir. Also, I think the purpose of checkin if the MAGENTO_ROOT . '/app/Mage.php' is useless, one file check for every request while magento wouldn't work at all if it didn't exist, I think that check has to be removed, there's no point in executing it.