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.
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
Add all properties when creating a subscription #25318
Conversation
@@ -1079,22 +1079,27 @@ function createSubscription($principalUri, $uri, array $properties) { | |||
'lastmodified' => time(), | |||
]; | |||
|
|||
foreach($this->subscriptionPropertyMap as $xmlName=>$dbName) { | |||
if (isset($properties[$xmlName])) { | |||
$propertiesBoolean = array('striptodos', 'stripalarms', 'stripattachments'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use short array declaration []
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed and squashed commits.
@georgehrke please test again - thx |
ab562ad
to
126f256
Compare
looks good to me 👍 |
👍 |
* Revert "Merge pull request #25240 from owncloud/remove-svg" This reverts commit 8b8d2b6, reversing changes made to a35747b. * Revert "Merge pull request #25253 from owncloud/users-fixotherquotadropdown" This reverts commit a35747b, reversing changes made to a573b68. * Revert "Merge pull request #25314 from owncloud/files_external-backends-config" This reverts commit a573b68, reversing changes made to 8147eef. * Revert "Add all properties while creating a subscription (#25318)" This reverts commit aaf4c30. * Revert "Merge pull request #25276 from owncloud/delete-own-session-token" This reverts commit e42ce62, reversing changes made to aaf4c30. * Revert "Merge pull request #25262 from owncloud/fed-sharing-error" This reverts commit 027715f, reversing changes made to e42ce62.
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Originally at #25231
Fix #24469