Improve photo and timezone sync scheduled task

[weilai-irl]

• Better support for site not using UPN as username,
• Reduce DB queries.

[Copilot]

Pull request overview

This PR improves the photoandtimezonesync scheduled task to better handle environments where o365name is not a valid Microsoft Graph user identifier, and to reduce per-user DB/API work during photo sync runs.

Changes:

• Use Entra objectid (GUID) as the identifier for Graph photo batch requests instead of o365name/UPN.
• Reduce DB overhead by replacing per-user local_o365_appassign lookups with a pre-aggregated LEFT JOIN, and by caching/storing a raw-photo hash in user_preferences to skip unnecessary image processing.
• Refine photo-apply semantics by returning a boolean from apply_photo_public() and improving change detection in apply_photo().

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
local/o365/classes/task/photoandtimezonesync.php	Switch photo batch lookup to objectid, reduce DB queries, and add hash-based short-circuiting for unchanged photos.
local/o365/classes/feature/usersync/main.php	Improve photo-apply “changed” detection and return a boolean from apply_photo_public() for scheduled task use.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.