Mark extra apps in non-writable folders as non-removeable and non-updateable

[boomer41]

How to use GitHub

• Please use the 👍 reaction to show that you are interested into the same feature.
• Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
• Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
I am building my own docker image of Nextcloud, extending the image from github.com/nextcloud/docker.

I am adding extra apps in a new folder additional_bundled_apps and I have added this folder in the configuration like this:

<?php
$CONFIG = array (
  'apps_paths' => array (
      0 => array (
              'path'     => OC::$SERVERROOT.'/apps',
              'url'      => '/apps',
              'writable' => false,
      ),
      1 => array (
              'path'     => OC::$SERVERROOT.'/additional_bundled_apps',
              'url'      => '/additional_bundled_apps',
              'writable' => false,
      ),
      2 => array (
              'path'     => OC::$SERVERROOT.'/custom_apps',
              'url'      => '/custom_apps',
              'writable' => true,
      ),
  ),
);

As you can see, I marked the new folder as non-writable. However, the App Store shows that I could uninstall the app.
Clicking on that button removes the entry temporarily, but upon pressing F5 the entry is back in the same state as before.
So that button click is a noop, which makes sense, because I marked that app path as non-writable.

However, apps residing in a non-writable folder should also be marked as non-removable to not even show that message. Also, updates should IMO be disabled because the server cannot write to that path.

Describe the solution you'd like
Apps residing in non-writable apps folders should also be marked and treated as non-removable and non-updatable.

Describe alternatives you've considered
We considered not bundling those apps, but then we'd have to rely on an internet connection when updating or bootstraping an instance.

Additional context
Running NC 30.0.0

[susnux]

bundled is something different, that are the core apps of Nextcloud included in the release and not published on the appstore.

I think this rather are two bugs:

• apps in read-only app directories should not be listed as remove-able
• apps in read-only app directories should not be able to update
  • (but probably still show there is an update if the appstore is enabled so you get the notification and can do it manually)

[boomer41]

I agree, the term "bundled" is a bit misleading on my side.
Your approach is exactly what I was asking. :)

EDIT: I have updated the description above.