Project

General

Profile

Actions

Bug #82068

closed

Install tool wizard Migrate all file relations from fe_user not visible

Added by Bernhard Eckl over 6 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Install Tool
Target version:
-
Start date:
2017-08-09
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

I upgraded from 7.6.16 to 8.7.4 but in the install tool I can not find the upgrade wizard: Migrate all file relations from fe_user.
See: https://docs.typo3.org/typo3cms/extensions/core/8.7/Changelog/8.3/Breaking-74375-Fe_usersimageMigratedToFAL.html


Files


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #81180: Frontend User upgrade wizard has flaky "done" checkClosedMarkus Klein2017-05-10

Actions
Actions #1

Updated by Wouter Wolters over 6 years ago

  • Status changed from New to Needs Feedback

The update wizard is named as FrontendUserImageUpdateWizard, can you find that class?

Actions #2

Updated by Bernhard Eckl over 6 years ago

Sure, I did not change the core… but the wizard is not shown in the install tool. Is there a condition for that?

Actions #4

Updated by Bernhard Eckl over 6 years ago

I finally got it migrated now. The following things I had to do:

- in public function checkForUpdate, line 138:

    return true;

otherwise the wizard would not be shown at all

- in protected function migrateField, line 298:
comment the line

$file = $this->storage->getFile($this->targetPath . $item);

and instead

$resourceFactory = \TYPO3\CMS\Core\Resource\ResourceFactory::getInstance();
$storage1 = $resourceFactory->getStorageObject(1);
$file = $storage1->getFile($this->targetPath . $item);

Otherwise no files where found.

Btw $storageUid = (int)$this->storage->getUid(); alwas returned 2, but it should be 1 (1 is my public storage named files, not fileadmin, 2 is a non public storage).

It seems that this migration wizard is buggy. Is there noone else having such difficulties with it?

Actions #5

Updated by Alexander Opitz over 6 years ago

  • Status changed from Needs Feedback to New
Actions #6

Updated by Online Now! GmbH over 5 years ago

I can confirm this behaviour. This is related to #81180 and the "wizard is done" check changed in that issues. It always returns false. After reverting this change, i got the wizard visible again.

Also when getting the file storages from the system, the wizards assumes the default fileadmin storage is the first. This wasnt the case in my installation. I had to change the line from "$this->storage = $storages0;" to "$this->storage = $storages1;". So either a better check for fileadmin storage should be implemented or let the user choose the storage.

Actions #7

Updated by Markus Klein over 5 years ago

  • Status changed from New to Accepted
  • Assignee set to Markus Klein
  • Target version set to next-patchlevel
  • Complexity set to easy
Actions #8

Updated by Markus Klein over 5 years ago

  • Related to Bug #81180: Frontend User upgrade wizard has flaky "done" check added
Actions #9

Updated by Gerrit Code Review over 5 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58895

Actions #10

Updated by Markus Klein over 5 years ago

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

Updated by Bernhard Eckl over 5 years ago

I just tried the update again, wizard is shown, but files are not migrated:
File 'uploads/pics/file.jpg' does not exist. Referencing field: fe_users.15.image. The reference was not migrated.

But the files exist.

Actions #12

Updated by Markus Klein over 5 years ago

@Eckl please create a new ticket for this issue. Thanks

Actions #13

Updated by Markus Klein over 5 years ago

@Eckl I retested this on current v8 branch and it worked. Please test again with the latest version from git and create a new ticket if it really does not work. Thank you.

Actions #14

Updated by Bernhard Eckl over 5 years ago

I just migrated the images manually (some images where successful by the updater, the other i have done myself). I think it was because of different sha1 for the images. Sorry, I didn’t have time to wait for a fix for another year since TYPO3 7 is not supported anymore.

Actions #15

Updated by Markus Klein over 5 years ago

  • Target version deleted (next-patchlevel)

Well, that's what I call supportive attitude. I was willing to dig into your problem, but fair enough.

Just to stick to the truth: I came to this ticket 28 days ago, not a year. Other than that you could have dug yourself and provide a patch. Just saying.

Actions #16

Updated by Bernhard Eckl over 5 years ago

I digged into the code and if I could solve it I would have provided a patch, but I couldn’t solve it. I think it is because I have different sha1 saved in sys_file. Some images where updated, some not. So I think this is just on my system since nobody else has this problem. And because I had no time for this (TYPO3 7 is not supported anymore) and I have 1000 other things to do, I decided to fix it manually to get version 8 finally. Keep calm. Just saying.

Actions #17

Updated by Benni Mack almost 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF