Skip to content

Remove some deprecated calls from ShareByMailProvider.php#26775

Merged
skjnldsv merged 1 commit into
nextcloud:masterfrom
robottod:patch-1
Apr 27, 2021
Merged

Remove some deprecated calls from ShareByMailProvider.php#26775
skjnldsv merged 1 commit into
nextcloud:masterfrom
robottod:patch-1

Conversation

@robottod

Copy link
Copy Markdown
Contributor

This moves the execute calls to the executeQuery and executeUpdate calls
Resulting in less calls to deprecated functions

Signed-off-by: rtod robottod@protonmail.com

This moves the execute calls to the executeQuery and executeUpdate calls
Resulting in less calls to deprecated functions

Signed-off-by: rtod <robottod@protonmail.com>
@robottod

Copy link
Copy Markdown
Contributor Author

It seems my DCO fails. Is there something I need to do? As I used the editor in the web interface

@rullzer

rullzer commented Apr 26, 2021

Copy link
Copy Markdown
Member

It seems my DCO fails. Is there something I need to do? As I used the editor in the web interface

Heya @robottod! And welcome here as well.
No worries I set the DCO to manually pass!

@LukasReschke LukasReschke added the 4. to release Ready to be released and/or waiting for tests to finish label Apr 26, 2021
@skjnldsv skjnldsv merged commit b8cde2c into nextcloud:master Apr 27, 2021
@welcome

welcome Bot commented Apr 27, 2021

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants