Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Sep 18, 2024

Summary

Looked up deprecation date where missing.

Checklist

@susnux susnux added 3. to review Waiting for reviews technical debt labels Sep 18, 2024
@susnux susnux added this to the Nextcloud 31 milestone Sep 18, 2024
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

Can you also extend the regex to enforce this?
And please also backport what is possible.

@susnux susnux force-pushed the chore/add-deprecation-date branch from 62c3d47 to 96b8470 Compare September 19, 2024 22:01
* @return string
*/
public function generateTarget($itemSource, $shareWith, $exclude = null) {
public function generateTarget($itemSource, $shareWith) {

Check notice

Code scanning / Psalm

MoreSpecificImplementedParamType

Argument 2 of OCA\Files_Sharing\ShareBackend\File::generateTarget has the more specific type 'string', expecting 'false|string' as defined by OCP\Share_Backend::generateTarget
@susnux susnux force-pushed the chore/add-deprecation-date branch from 96b8470 to 777e349 Compare September 19, 2024 22:13
@susnux susnux force-pushed the chore/add-deprecation-date branch from 777e349 to e243cb8 Compare September 19, 2024 22:51
@susnux susnux merged commit 4ba3d4a into master Sep 20, 2024
@susnux susnux deleted the chore/add-deprecation-date branch September 20, 2024 01:03
@skjnldsv skjnldsv mentioned this pull request Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants