Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 20, 2025

Backport of #50542

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added bug 3. to review Waiting for reviews labels Mar 20, 2025
@backportbot backportbot bot added this to the Nextcloud 30.0.9 milestone Mar 20, 2025
@luka-nextcloud luka-nextcloud force-pushed the backport/50542/stable30 branch from 4c227c3 to 83e2606 Compare March 21, 2025 08:54
@luka-nextcloud luka-nextcloud marked this pull request as ready for review March 21, 2025 08:54
@provokateurin provokateurin enabled auto-merge March 21, 2025 09:00
@luka-nextcloud luka-nextcloud disabled auto-merge March 21, 2025 16:38
@AndyScherzinger AndyScherzinger force-pushed the backport/50542/stable30 branch from 83e2606 to d5c53a0 Compare March 22, 2025 16:12
@luka-nextcloud luka-nextcloud force-pushed the backport/50542/stable30 branch from d5c53a0 to fa0c906 Compare March 27, 2025 10:26
if ($share->getExpirationDate() === null
|| $share->getExpirationDate()->getTimestamp() < $minTime->getTimestamp()
|| $share->getExpirationDate()->getTimestamp() > $maxTime->getTimestamp()) {
|| $share->getExpirationDate()->getTimestamp() > $maxTime->getTimestamp()

Check notice

Code scanning / Psalm

PossiblyNullReference Note

Cannot call method getTimestamp on possibly null value
@luka-nextcloud luka-nextcloud force-pushed the backport/50542/stable30 branch from fa0c906 to 986a570 Compare March 27, 2025 11:56
@luka-nextcloud luka-nextcloud merged commit 713ead5 into stable30 Mar 27, 2025
181 of 183 checks passed
@luka-nextcloud luka-nextcloud deleted the backport/50542/stable30 branch March 27, 2025 16:24
@Altahrim Altahrim mentioned this pull request Apr 3, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants