Only set the default expiration date on share creation

Fixes #19685

The default expiration date should only be set when we create a new
share. So if a share is created and the expiration date is unset. And
after that the password is updated the expiration date should remain
unset.

3 files changed, 5 insertions, 2 deletions

diff --git a/lib/private/share20/manager.php b/lib/private/share20/manager.php
index 7cd44a7cb37..c00d7aa4072 100644
--- a/lib/private/share20/manager.php
+++ b/lib/private/share20/manager.php
@@ -261,7 +261,7 @@ class Manager implements IManager {
 		}
 
 		// If expiredate is empty set a default one if there is a default
-		if ($expirationDate === null && $this->shareApiLinkDefaultExpireDate()) {
+		if ($share->getFullId() === null && $expirationDate === null && $this->shareApiLinkDefaultExpireDate()) {
 			$expirationDate = new \DateTime();
 			$expirationDate->setTime(0,0,0);
 			$expirationDate->add(new \DateInterval('P'.$this->shareApiLinkDefaultExpireDays().'D'));
diff --git a/lib/private/share20/share.php b/lib/private/share20/share.php
index cd30f24c42e..eb112d37942 100644
--- a/lib/private/share20/share.php
+++ b/lib/private/share20/share.php
@@ -90,6 +90,9 @@ class Share implements \OCP\Share\IShare {
 	 * @inheritdoc
 	 */
 	public function getFullId() {
+		if ($this->providerId === null || $this->id === null) {
+			return null;
+		}
 		return $this->providerId . ':' . $this->id;
 	}
 
diff --git a/lib/public/share/ishare.php b/lib/public/share/ishare.php
index 5054a886af5..8f5175b0dbf 100644
--- a/lib/public/share/ishare.php
+++ b/lib/public/share/ishare.php
@@ -46,7 +46,7 @@ interface IShare {
 	 * Get the full share id. This is the <providerid>:<internalid>.
 	 * The full id is unique in the system.
 	 *
-	 * @return string
+	 * @return string|null null is returned when the fullId can't be constructed
 	 * @since 9.0.0
 	 */
 	public function getFullId();