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

Magento Connect Should Not Report Extensions are Installed if they're Not Installed #95

Closed
astorm opened this issue Oct 1, 2012 · 3 comments

Comments

@astorm
Copy link

astorm commented Oct 1, 2012

Current versions of the Magento Connect Package Management system will report that a package has been successfully installed even if Magento Connect was unable to install the package due to file permission errors.

Steps to Reproduce

  1. Find a Magento Connect package file that installs files to app/code/community
  2. Change file permissions on app/code/community such that the web server cannot write to app/code/community (i.e. chmod 755 app/code/community)
  3. Log into Magento Connect
  4. Click "Choose File"
  5. Browse to the Magento Connect package file and select it
  6. Click the "Upload" button

Expected Behavior

  • Magento Connect reports it was unable to install the package due to file permission errors

Actual Behavior

  • Magento Connect reports it was able to install the package and adds it to the list of packages
  • Package is not actually installed on system

A screencast demoing this problem has been uploaded to YouTube here

http://www.youtube.com/watch?v=TmRxTTmGyN0

@magento-team
Copy link
Contributor

I reported it to our bugtracker however submitting a fix with code and test would highly increase the chance of such issues being fixed in Magento core.

@astorm
Copy link
Author

astorm commented Oct 1, 2012

This fix requires feedback from product/leads on the best direction to take, and probably requires architecture changes to Magento connect itself to accommodate the fix. I get wanting to encourage contributions, but a generic copy/paste "fix with code and test" comes off as tone-deaf here.

@magento-team magento-team reopened this Oct 2, 2012
@magento-team
Copy link
Contributor

Hello Alan, we expect major changes to happen in the connect itself so let's leave this one for now as a bug report.

magento-team pushed a commit that referenced this issue Feb 20, 2015
[Troll] Varnish updates and Unit Tests Coverage
magento-team pushed a commit that referenced this issue Nov 3, 2015
[EPAM] Product Video. Bugfixing. PR5
okorshenko pushed a commit that referenced this issue Jun 12, 2016
magento-team pushed a commit that referenced this issue Jan 10, 2018
…e order fixed #95

 - Merge Pull Request magento-engcom/import-export-improvements#95 from umarch-rltsquare/magento2:71-reverse-export-custom-options-fix
 - Merged commits:
   1. e1b5040
   2. 558f714
magento-engcom-team pushed a commit that referenced this issue Apr 13, 2018
MAGETWO-89321: Misconfigured composer.json and composer.lock
magento-engcom-team pushed a commit to okorshenko/magento2 that referenced this issue May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants