mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #26845 from nextcloud/fix/noid/import-card-fix-string-value 

fix creating vcards with multiple string values
This commit is contained in:
Morris Jobke 2021-05-03 15:49:43 +02:00 committed by GitHub
parent 7c3f8331ef a9f39aab29
commit 68c8fa8235
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 23 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
apps/dav/lib/CardDAV/AddressBookImpl.php
View File

@ -150,6 +150,9 @@ class AddressBookImpl implements IAddressBook {
if (is_array($value)) { if (is_array($value)) {
$vCard->remove($key); $vCard->remove($key);
foreach ($value as $entry) { foreach ($value as $entry) {
if (is_string($entry)) {
$property = $vCard->createProperty($key, $entry);
} else {
if (($key === "ADR" || $key === "PHOTO") && is_string($entry["value"])) { if (($key === "ADR" || $key === "PHOTO") && is_string($entry["value"])) {
$entry["value"] = stripslashes($entry["value"]); $entry["value"] = stripslashes($entry["value"]);
$entry["value"] = explode(';', $entry["value"]); $entry["value"] = explode(';', $entry["value"]);
@ -158,6 +161,7 @@ class AddressBookImpl implements IAddressBook {
if (isset($entry["type"])) { if (isset($entry["type"])) {
$property->add('TYPE', $entry["type"]); $property->add('TYPE', $entry["type"]);
} }
}
$vCard->add($property); $vCard->add($property);
} }
} elseif ($key !== 'URI') { } elseif ($key !== 'URI') {

14
apps/dav/tests/unit/CardDAV/AddressBookImplTest.php
View File

@ -154,11 +154,20 @@ class AddressBookImplTest extends TestCase {
->setMethods(['vCard2Array', 'createUid', 'createEmptyVCard']) ->setMethods(['vCard2Array', 'createUid', 'createEmptyVCard'])
->getMock(); ->getMock();
$expectedProperties = 0;
foreach ($properties as $data) {
if (is_string($data)) {
$expectedProperties++;
} else {
$expectedProperties += count($data);
}
}
$addressBookImpl->expects($this->once())->method('createUid') $addressBookImpl->expects($this->once())->method('createUid')
->willReturn($uid); ->willReturn($uid);
$addressBookImpl->expects($this->once())->method('createEmptyVCard') $addressBookImpl->expects($this->once())->method('createEmptyVCard')
->with($uid)->willReturn($this->vCard); ->with($uid)->willReturn($this->vCard);
$this->vCard->expects($this->exactly(count($properties))) $this->vCard->expects($this->exactly($expectedProperties))
->method('createProperty'); ->method('createProperty');
$this->backend->expects($this->once())->method('createCard'); $this->backend->expects($this->once())->method('createCard');
$this->backend->expects($this->never())->method('updateCard'); $this->backend->expects($this->never())->method('updateCard');
@ -172,7 +181,8 @@ class AddressBookImplTest extends TestCase {
public function dataTestCreate() { public function dataTestCreate() {
return [ return [
[[]], [[]],
[['FN' => 'John Doe']] [['FN' => 'John Doe']],
[['FN' => 'John Doe', 'EMAIL' => ['john@doe.cloud', 'john.doe@example.org']]],
]; ];
} }