Skip to content

Adding isWatermarked to BoxItem.#371

Merged
dmaynard merged 6 commits intomasterfrom
add_watermark_info
Nov 29, 2016
Merged

Adding isWatermarked to BoxItem.#371
dmaynard merged 6 commits intomasterfrom
add_watermark_info

Conversation

@dmaynard
Copy link
Copy Markdown
Contributor

No description provided.

@boxcla
Copy link
Copy Markdown

boxcla commented Nov 17, 2016

Verified that @dmaynard has signed the CLA. Thanks for the pull request!

Copy link
Copy Markdown
Contributor

@gcurtis gcurtis left a comment

Choose a reason for hiding this comment

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

Just a minor fix is needed with the isWatermarked field.

Comment thread src/main/java/com/box/sdk/BoxItem.java Outdated
* Gets flag indicating whether this file is Watermarked.
* @return whether the file is watermarked or not
*/
public Boolean getIsWatermarked() {
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.

This should be lowercase boolean.

Comment thread src/main/java/com/box/sdk/BoxItem.java Outdated
private BoxFolder.Info parent;
private String itemStatus;
private Set<BoxCollection.Info> collections;
private Boolean isWatermarked;
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.

Lowercase boolean to use the primitive datatype.

Comment thread src/main/java/com/box/sdk/BoxFile.java Outdated
* Gets flag indicating whether this file is Watermarked.
* @return whether the file is watermarked or not
*/
public Boolean getIsWatermarked() {
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.

This also needs to be lowercase boolean.

* Gets flag indicating whether this file is Watermarked.
* @return whether the file is watermarked or not
*/
public Boolean getIsWatermarked() {
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.

This also needs to be lowercase boolean.

@dmaynard dmaynard merged commit d14b7ce into master Nov 29, 2016
@dmaynard
Copy link
Copy Markdown
Contributor Author

Changed isWatermarked from Boolean to boolean

@mattwiller mattwiller deleted the add_watermark_info branch December 15, 2018 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants