Skip to content

add "askForPassword" on capabilities for sharing - #15461

Merged
MorrisJobke merged 1 commit into
masterfrom
askForPasswordCapability
May 13, 2019
Merged

add "askForPassword" on capabilities for sharing#15461
MorrisJobke merged 1 commit into
masterfrom
askForPasswordCapability

Conversation

@tobiasKaminsky

Copy link
Copy Markdown
Member

Needed for #15459

This exposes "askForPassword" capability on file_sharing:
If password is enforced, askForPassword will be false.

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

@tobiasKaminskytobiasKaminsky added the 3. to review Waiting for reviews label May 9, 2019
Comment threadapps/files_sharing/lib/Capabilities.php Outdated
Comment threadapps/files_sharing/lib/Capabilities.php Outdated

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

fine by me

@kesselbkesselb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. We could save another call here by reusing $public['password']['enforced'].

Comment threadapps/files_sharing/lib/Capabilities.php Outdated
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@MorrisJobke
MorrisJobkeforce-pushed the askForPasswordCapability branch from 0df2056 to b65a595CompareMay 13, 2019 18:09
@MorrisJobkeMorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 13, 2019
@MorrisJobke
MorrisJobke merged commit 8001c89 into masterMay 13, 2019
@MorrisJobke
MorrisJobke deleted the askForPasswordCapability branch May 13, 2019 19:17
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishenhancementfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@tobiasKaminsky@rullzer@MorrisJobke@kesselb