Skip to content

feat: Document IAppManager::getAppNamespace new method#14799

Open
come-nc wants to merge 1 commit intomasterfrom
feat/document-iappmanager-getappnamespace
Open

feat: Document IAppManager::getAppNamespace new method#14799
come-nc wants to merge 1 commit intomasterfrom
feat/document-iappmanager-getappnamespace

Conversation

@come-nc
Copy link
Copy Markdown
Contributor

@come-nc come-nc commented May 7, 2026

☑️ Resolves

🖼️ Screenshots

✅ Checklist

  • I have built the documentation locally and reviewed the output
  • Screenshots are included for visual changes
  • I have not moved or renamed pages (or added a redirect if I did)
  • I have run codespell or similar and addressed any spelling issues

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc added this to the Nextcloud 34 milestone May 7, 2026
@come-nc come-nc self-assigned this May 7, 2026
@come-nc come-nc requested review from ChristophWurst and susnux and removed request for ChristophWurst May 7, 2026 13:42
Comment on lines +41 to +45
``\OCP\App\IAppManager`` was extended with two new methods related to application namespaces:
- ``getAppNamespace(string $appId): string`` returns the namespace for an application from its appid
- ``getAppFromNamespace(string $className): ?string`` does the opposite. Less common but it is used in guests application.

This replaces a static method in ``\OCP\AppFramework\App`` which is now deprecated.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code needs a dedicated section, not just the announcement. We will clear the New in this release page after branch-off. See the other items on this page.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no existing IAppManager documentation, so one has to be started. I suggest to put it under https://docs.nextcloud.com/server/latest/developer_manual/app_development/index.html.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not understand what this means.
The existing IAppManager documentation is at https://nextcloud-server.netlify.app/classes/ocp-app-iappmanager
Linked in https://docs.nextcloud.com/server/latest/developer_manual/digging_deeper/api.html#php-public-api

Why would we duplicate this in this repository?

I thought that after branch-off this would be moved to release-notes/upgrade_to_34.rst, what is the actual process?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The class reference is great for diving into details, but it doesn't tell you how and why a class should be used.
Not every app developer is a senior with years of experience with Nextcloud. They don't know how things work just by looking at documentation of classes.

This section was recently reorganized because we only had the diff and not the current state. New will not be moved into a page. We want to get away from this heap of diffs to a comprehensive manual for developers.

I'll try to document this goal better.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

📖 Documentation Preview

🔍 Open preview →

📄 2 changed documentation pages

Last updated: Thu, 07 May 2026 16:54:11 GMT

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants