Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Get access list share by email recipients #32631

Merged
merged 5 commits into from
Mar 25, 2024
Merged

Get access list share by email recipients #32631

merged 5 commits into from
Mar 25, 2024

Commits on Mar 21, 2024

  1. ShareByMailProvider: getAccessList: return full list of recipients 

    Previously was returning only boolean true if the Node was shared
by email, or false if not. Now provides an array containing the
list of email share recipients.

Signed-off-by: Adam Serbinski <adam@serbinski.com>
    @ASerbinski
    ASerbinski committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    26ae5e6 View commit details
    Browse the repository at this point in the history

  2. ShareByMailProvider: getAccessList: add token to returned array 

    This allows the share URI to be regenerated.

Signed-off-by: Adam Serbinski <adam@serbinski.com>
    @ASerbinski
    ASerbinski committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    9c528d4 View commit details
    Browse the repository at this point in the history

  3. ShareByMailProvider: getAccessList: recommended style changes 

    Signed-off-by: Adam Serbinski <adam@serbinski.com>
    @ASerbinski
    ASerbinski committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    34ca6bd View commit details
    Browse the repository at this point in the history

  4. ShareByMailProvider: getAccessList: set both array keys 'public' and … 

    …'mail'

To preserve prior behaviour where 'public' was set 'true' if there
are any mail recipients. The 'mail' key will be an array of email
recipients.

Signed-off-by: Adam Serbinski <adam@serbinski.com>
    @ASerbinski
    ASerbinski committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    01e3234 View commit details
    Browse the repository at this point in the history

  5. ShareByMailProvider: getAccessList: set correct value for 'public' 

    PDOStatement::rowCount behavior is undefined for SELECT statements
for some database types, therefore manually set the value for 'public'
based on actual results fetched.

Signed-off-by: Adam Serbinski <adam@serbinski.com>
    @ASerbinski
    ASerbinski committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    f45eb75 View commit details
    Browse the repository at this point in the history