Bug #66410

EM: Dependency errors are lost as a side-effect of sub-dependency check

Added by Xavier Perseguers over 7 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Extension Manager
Target version:
Start date:
2015-04-14
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

How to reproduce

  • Tested on TYPO3 6.2
  • Download (without installing) TemplaVoil√† 1.9.0
  • (Possibly download static info tables as well but without installing)
  • Make sure your list of extensions is up to date
  • Refresh EM
  • It shows that TemplaVoil√† may be updated and a click on the icon shows it will try to use version 2.0.3 instead of 1.9.7

Important

  • This only happens when neither TV nor SIT are loaded
  • If #65117 is applied, then after activating/loading SIT, TV shows an upgrade to 1.9.7 only, even when not yet loaded

Related issues

Related to TYPO3 Core - Bug #65117: Extensionmanager: Update of Extension with higher TYPO3-dependency not possibleClosedAlexander Opitz2015-02-17

Actions
Has duplicate TYPO3 Core - Bug #66414: Depency errors not savedClosedNicole Cordes2015-04-14

Actions
#1

Updated by Xavier Perseguers over 7 years ago

  • Status changed from New to Accepted

Problem when SIT is not loaded is that \TYPO3\CMS\Extensionmanager\Utility\DependencyUtility::checkDependencies resets the list of errors when it checks another extension. So when TV is checked, it checks for dependency errors of its own dependencies and this resets the list of problems!

#2

Updated by Xavier Perseguers over 7 years ago

To be more precise, checkDependencies finds an error with TV version 2.0.3 (check on TYPO3 Core), then searches for problems with SIT, which in turns calls \TYPO3\CMS\Extensionmanager\Service\ExtensionManagementService::markExtensionForInstallation() with SIT as parameter and this one calls again checkDependencies which resets the list of problems.

#3

Updated by Xavier Perseguers over 7 years ago

Furthermore, if fixed but without #65117 applied, the (other) problem remains: the update button is not visible for TV

#4

Updated by Xavier Perseguers over 7 years ago

  • Target version set to 6.2.12
  • Sprint Focus set to Stabilization Sprint
#5

Updated by Xavier Perseguers over 7 years ago

  • Subject changed from Extension Manager shows incompatible version as potential update to EM: Dependency errors are lost as a side-effect of sub-dependency check
#6

Updated by Gerrit Code Review over 7 years ago

  • Status changed from Accepted to Under Review

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38686

#7

Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38688

#8

Updated by Nicole Cordes over 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#9

Updated by Riccardo De Contardi almost 5 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF