Skip to content

Fix undefined share object during expirationDate update - #43661

Merged
nfebe merged 2 commits into
masterfrom
43256-fix-update-share-expiration-date
Feb 20, 2024
Merged

Fix undefined share object during expirationDate update#43661
nfebe merged 2 commits into
masterfrom
43256-fix-update-share-expiration-date

Conversation

@nfebe

Copy link
Copy Markdown
Contributor

In 04e8733 a regression was introduced. this means nothing inside arrow functions, hence this.share was referring to an undefined object and so all attempted updates for expiration dates would fail.

Resolves : #43256

@emoral435emoral435 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! 🐘

In 04e8733 a regression was introduced.
`this` means nothing inside arrow functions, hence this.share was referring to
an undefined object and so all attempted updates for expiration dates would fail.
Resolves : #43256
Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
@nfebe
nfebeforce-pushed the 43256-fix-update-share-expiration-date branch from 92e8cc1 to 82cdc10CompareFebruary 20, 2024 10:10
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/compile /

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nfebe
nfebe merged commit d77510c into masterFeb 20, 2024
@nfebe
nfebe deleted the 43256-fix-update-share-expiration-date branch February 20, 2024 11:49
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/backport to stable28

@nfebe

Copy link
Copy Markdown
ContributorAuthor

/backport to stable27

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

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[Bug]: Unable to update share expiration date

4 participants

@nfebe@Altahrim@emoral435@nextcloud-command