Skip to content

Conversation

@ArtificialOwl
Copy link
Member

The idea is to run a single command from CirclesManager in a super session:

		/** @var CirclesManager $manager */
		$manager = \OC::$server->get(CirclesManager::class);
		$result = $manager->runAsSuperSession('getCircles', [new CircleProbe()]);

@ArtificialOwl ArtificialOwl force-pushed the enh/noid/run-as-super-session branch from ac8c37a to b34f273 Compare March 29, 2022 11:58
@ArtificialOwl ArtificialOwl requested a review from mejo- March 29, 2022 11:58
@ArtificialOwl ArtificialOwl force-pushed the enh/noid/run-as-super-session branch 2 times, most recently from 57b333f to bacc3d9 Compare March 29, 2022 12:03
Signed-off-by: Maxence Lange <[email protected]>
@ArtificialOwl ArtificialOwl force-pushed the enh/noid/run-as-super-session branch from bacc3d9 to ba44a8c Compare April 1, 2022 11:57
* Run a single method as Super, then Super level will be removed
*
* @throws Throwable
* @noinspection PhpUndefinedVariableInspection
Copy link
Member

Choose a reason for hiding this comment

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

This sounds scary, undefined variables will throw errors in PHP 8.2

Copy link
Member Author

Choose a reason for hiding this comment

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

Returned is mixed
let's put this in standby, merge is not necessary.

@blizzz blizzz added this to the Nextcloud 30 milestone Apr 11, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 22, 2024
44 tasks
@blizzz blizzz modified the milestones: Nextcloud 30, Nextcloud 31 Sep 4, 2024
@blizzz blizzz mentioned this pull request Jan 29, 2025
1 task
@blizzz blizzz modified the milestones: Nextcloud 31, Nextcloud 32 Jan 29, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
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.

5 participants