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

reenable shipped apps is not working #301

Closed
DeepDiver1975 opened this issue Apr 6, 2016 · 9 comments
Closed

reenable shipped apps is not working #301

DeepDiver1975 opened this issue Apr 6, 2016 · 9 comments
Labels
Milestone

Comments

@DeepDiver1975
Copy link
Member

  • the array $this->disabledApps is never set
  • even if - how is the state preserved between two invocations of the upgrade application?
@DeepDiver1975 DeepDiver1975 added this to the 9.1-current milestone Apr 6, 2016
@VicDeo
Copy link
Member

VicDeo commented Apr 7, 2016

the array $this->disabledApps is never set

true

even if - how is the state preserved between two invocations of the upgrade application?

it is serialized into the session:
https://github.com/owncloud/updater/blob/master/src/Command/DisableNotShippedAppsCommand.php#L42

@DeepDiver1975
Copy link
Member Author

it is serialized into the session:

using the session in a cli command?

@DeepDiver1975
Copy link
Member Author

reoping until open question is answered

@Croydon
Copy link

Croydon commented Apr 8, 2016

Please fix this. It is really annoying to enable apps after every single update.

@Croydon
Copy link

Croydon commented Apr 8, 2016

Discussion: owncloud/core#18312

@VicDeo
Copy link
Member

VicDeo commented Apr 9, 2016

@DeepDiver1975 yes, for CLI too

@DeepDiver1975
Copy link
Member Author

This was not a serious question. Using sessions in cli mode is non sense

@VicDeo
Copy link
Member

VicDeo commented Apr 18, 2016

@DeepDiver1975 ok, then it uses Registry singleton in the scope of upgrade:start command to obtain persistence in CLI mode https://github.com/owncloud/updater/blob/master/src/Command/StartCommand.php

@VicDeo
Copy link
Member

VicDeo commented May 4, 2016

@DeepDiver1975 tested migration stable9->master with
#307
#346
#347
applied

all apps (shipped/not shipped) are being reenabled correctly

@VicDeo VicDeo closed this as completed May 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants