feat(admin): Document server autoloading and class maps #12024
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
miaulalala
reviewed
Jul 17, 2024
| When you add, move or delete a class file you have to update the autoloaders for the class to be found. The process is slightly different for classes that belong to the server itself, and apps that are included in the server repository: | ||
|
|
||
| 1. Use ``composer dump-autoload`` to update the server class map. | ||
| 2. Use ``composer -d apps/dav/composer dump-autoload`` to update the class map of the dav app. |
There was a problem hiding this comment.
Maybe add a sentence that this applies to all apps that are included?
Suggested change
| 2. Use ``composer -d apps/dav/composer dump-autoload`` to update the class map of the dav app. | |
| 2. Use ``composer -d apps/dav/composer dump-autoload`` to update the class map of the dav app, for example. All shipped apps have their own composer folder where you should run this command. |
There was a problem hiding this comment.
Or run bash build/autoloaderchecker.sh && git restore apps/**/installed.php to update all
There was a problem hiding this comment.
Works too, but is very slow. And with the many lines printed I have the feeling most engineers don't know what they are doing. So I prefer the more direct approach.
miaulalala
approved these changes
Jul 17, 2024
Altahrim
approved these changes
Aug 6, 2024
Signed-off-by: Christoph Wurst <[email protected]>
ChristophWurst
force-pushed
the
feat/admin/server-autoloading
branch
from
c8ffef8 to
ae9bc1e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
☑️ Resolves
🖼️ Screenshots