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

Remove files_move from appstore #26503

Closed
ghost opened this issue Oct 27, 2016 · 5 comments
Closed

Remove files_move from appstore #26503

ghost opened this issue Oct 27, 2016 · 5 comments

Comments

@ghost
Copy link

ghost commented Oct 27, 2016

Hi,

the files_move app (https://apps.owncloud.com/content/show.php/Files+move?content=150271) is known to cause various bugs like explained at:

owncloud/gallery#701, owncloud/gallery#699, owncloud/gallery#702, eotryx/oc_files_mv#29 (comment), eotryx/oc_files_mv#27 (and some other issues).

As the app developer seems to not getting active to include an already provided fix: eotryx/oc_files_mv#29 i would suggest to remove that app from the appstore for now. This prevents that people are installing that app on newer oC versions and breaking their instance with that.

One additional issue is that the app itself is not enforcing an upper version in its info.xml:

https:/eotryx/oc_files_mv/blob/master/appinfo/info.xml#L13

so oC9.1 user are e.g. also getting updates which breaks gallery, documents and maybe also other apps and functionalities of oC.

@PVince81
Copy link
Contributor

@DeepDiver1975 @felixboehm what do you think ?

@PVince81
Copy link
Contributor

Doesn't the app store also have a field to set max version, maybe if we just set the value there the app store will stop delivering the app for future versions ?

On the other hand people who already have the app enabled might still suffer from the problem due to incomplete info.xml

@ghost
Copy link
Author

ghost commented Oct 28, 2016

The files_mv app has set a maximum required: ownCloud 9.0 but that doesn't solve the issue that:

  • people who already have the app enabled might still suffer from the problem due to incomplete info.xml
  • people are still able to download and activate that app from the appstore manually due to the incomplete info.xml

@DeepDiver1975
Copy link
Member

As first action I remove the approved state of the app - it will be listed as experimental app for now.

@PVince81
Copy link
Contributor

I guess that should be enough for now, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants