Project

General

Profile

Actions

Bug #21499

closed

Update Wizard step 3: Uninstalling all new system extensions outputs "Update FAILED!"

Added by Daniel Doesburg over 14 years ago. Updated about 13 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2009-11-09
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.4
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Update Wizard step 3: Perform updates outputs "Update FAILED!".
This only occurs when I want to uninstall all "installSystemExtensions". If I uncheck all chackboxes I get those message.

And all other things are fine.

(issue imported from #M12511)


Files


Related issues 1 (0 open1 closed)

Is duplicate of TYPO3 Core - Bug #24583: Not installing extensions in Upgrade Wizard should also get rid of the wizardClosedErnesto Baschny2011-01-26

Actions
Actions #1

Updated by Chris topher almost 14 years ago

Is that still a problem?

At step 3 a database connection is needed and the file localconf.php needs to be writable. If it is still a problem, please check, that both is the case.

Actions #2

Updated by Benni Mack almost 14 years ago

Yes. Please try out the latest beta (4.4beta2) and let us know if still fails.

Actions #3

Updated by Daniel Doesburg almost 14 years ago

With 4.4.0b3 still the same. But only if I want to uninstall all "installSystemExtensions". If I uncheck all chackboxes I get those message. And all other things are fine. Database compare is no problem. Whole site acts as it should. Only this is a little cosmetic problem, because I can of course change all the extensions in the EM. But for newbies it's confusing

Actions #4

Updated by Francois Suter over 13 years ago

I agree with this problem. To rephrase: if you are in one of the 2 wizards that propose to install extensions (outsourced ones or news ones) and you uncheck all the boxes, then proceed with the update, the result - as displayed on the next screen - will be "Update FAILED". This is not due to any code error. It's due to the fact that performUpdate() returns FALSE when it does nothing.

The problem is that currently performUpdate() is keyed to return a boolean value, so we have only TRUE and FALSE as available return status. We would need to change that to include "skipped" status IMO. This is too late now for 4.5, but it should be done for 4.6.

Actions #5

Updated by Steffen Gebert over 13 years ago

Why not return TRUE, even if the user decided to install no extension?

Actions #6

Updated by Francois Suter over 13 years ago

This could be quick fix, but I would find it better to have a specific status for skipping an update. I think it's cleaner. A skipped update is not really successful. It would also make it possible to display a different confirmation message, maybe hinting at how one should be careful when skipping an update.

Actions #7

Updated by Steffen Gebert over 13 years ago

Of course, there should be a skipped state. However, from the user sight I would say that the wizard was successful - it did, what the user asked to do.

Skipped wizards should also remember their state and never show up again.

Actions #8

Updated by Ernesto Baschny about 13 years ago

Both issues solved in #24583. Please test (pending)

Actions #9

Updated by Ernesto Baschny about 13 years ago

Fixed in #24583

Actions #10

Updated by Steffen Gebert about 13 years ago

Steffen, why reopened?

Actions #11

Updated by Susanne Moog about 13 years ago

  • Target version deleted (4.5.0)
Actions

Also available in: Atom PDF