Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

+runAsSuperSession #980

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

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 <maxence@artificial-owl.com>
@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 modified the milestones: Nextcloud 27, Nextcloud 28 May 23, 2023
@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
@skjnldsv skjnldsv mentioned this pull request Mar 28, 2024
81 tasks
@blizzz blizzz modified the milestones: Nextcloud 29, Nextcloud 30 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
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