[10.2.0] Respect 'writable' appdir flag on update

[VicDeo]

Backport of #35072

Description

Check if the current appdir is marked as writable before updating the app

Related Issue

• Fixes App manager not respecting writable flag when folder writable on disk #35031

Motivation and Context

Fix App is updated inside the directory that is marked as not writable.

How Has This Been Tested?

see #35031

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Database schema changes (next release will require increase of minor version instead of patch)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

[PVince81]

👍

[codecov]

Codecov Report

Merging #35097 into release-10.2.0 will increase coverage by 0.01%.
The diff coverage is 91.66%.

@@                 Coverage Diff                  @@
##             release-10.2.0   #35097      +/-   ##
====================================================
+ Coverage             64.29%    64.3%   +0.01%     
- Complexity            20044    20049       +5     
====================================================
  Files                  1285     1285              
  Lines                 76846    76854       +8     
  Branches               1307     1307              
====================================================
+ Hits                  49410    49423      +13     
+ Misses                27055    27050       -5     
  Partials                381      381

Flag	Coverage Δ	Complexity Δ
#javascript	53.01% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	65.5% <91.66%> (+0.01%)	20049 <0> (+5)	⬆️

Impacted Files	Coverage Δ	Complexity Δ
lib/private/legacy/app.php	63.45% <100%> (+1.31%)	179 <0> (+5)	⬆️
lib/private/Installer.php	48.18% <66.66%> (ø)	85 <0> (ø)	⬇️
lib/private/AllConfig.php	95.89% <0%> (+1.36%)	43% <0%> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 73beb5d...ae83531. Read the comment docs.

[PVince81]

👍