Make CalDAV resource sync a service and expose as occ command

[ChristophWurst]

Summary

CalDAV resources and rooms are provided by apps. Currently the rooms and resources are read in a background job once an hour. Sometimes one wants to see the changes right away.

By moving the job logic into a reusable service we can also invoke it from an occ command.

This also drops the two unused events that aren't emitted anywhere

• \OCP\Calendar\Resource\ForceRefreshEvent
• \OCP\Calendar\Room\ForceRefreshEvent

TODO

• Move logic into a service
• Add occ command
• Documentation

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[github-advanced-security]

Psalm found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

[PVince81]

👍 makes sense

I guess we can't use the new occ background-job:execute instead ?

cc @come-nc

[come-nc]

I think using background-job:execute works too in this case, given the job is always planned.

But that requires using background-job:list first to get the id, so I have no real problem with having a specific command for it.
That said we should look into improving background-job:execute to be able to run jobs by class name, and background-job:list to be able so search for jobs without knowing the exact name.

[come-nc]

Please avoid silent commands, at least output something in verbose mode.

[come-nc]

Suggested change

	private CalendarResourcesRoomsSyncService $syncService;
	public function __construct(CalendarResourcesRoomsSyncService $syncService) {
	parent::__construct();
	$this->syncService = $syncService;
	}
	public function __construct(
	private CalendarResourcesRoomsSyncService $syncService,
	) {
	parent::__construct();
	}

Use the force Luke (if you do not need backports)