grommunio / grommunio-dav

lib/AuthBasicBackend.php

Braces +2 added lines, -4 removed lines

@@ -16,8 +16,7 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct(GrommunioDavBackend $gDavBackend)
-	{
+	public function __construct(GrommunioDavBackend $gDavBackend) {
 		$this->gDavBackend = $gDavBackend;
 	}
 
@@ -34,8 +33,7 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function validateUserPass($username, $password)
-	{
+	protected function validateUserPass($username, $password) {
 		return $this->gDavBackend->Logon($username, $password);
 	}
 }

lib/GrommunioCardDavBackend.php

Braces +12 added lines, -24 removed lines

@@ -21,8 +21,7 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct(GrommunioDavBackend $gDavBackend, GLogger $glogger)
-	{
+	public function __construct(GrommunioDavBackend $gDavBackend, GLogger $glogger) {
 		$this->gDavBackend = $gDavBackend;
 		$this->logger = $glogger;
 	}
@@ -45,8 +44,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function getAddressBooksForUser($principalUri)
-	{
+	public function getAddressBooksForUser($principalUri) {
 		$this->logger->trace("principalUri: %s", $principalUri);
 
 		return $this->gDavBackend->GetFolders($principalUri, static::CONTAINER_CLASSES);
@@ -66,8 +64,7 @@ 
 	 *
 	 * @param string $addressBookId
 	 */
-	public function updateAddressBook($addressBookId, \Sabre\DAV\PropPatch $propPatch)
-	{
+	public function updateAddressBook($addressBookId, \Sabre\DAV\PropPatch $propPatch) {
 		// TODO is our logger able to log this object? It probably needs to be adapted.
 		$this->logger->trace("addressBookId: %s - proppatch: %s", $addressBookId, $propPatch);
 	}
@@ -83,8 +80,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function createAddressBook($principalUri, $url, array $properties)
-	{
+	public function createAddressBook($principalUri, $url, array $properties) {
 		$this->logger->trace("principalUri: %s - url: %s - properties: %s", $principalUri, $url, $properties);
 		// TODO Add displayname
 		return $this->gDavBackend->CreateFolder($principalUri, $url, static::CONTAINER_CLASS, "");
@@ -95,8 +91,7 @@ 
 	 *
 	 * @param mixed $addressBookId
 	 */
-	public function deleteAddressBook($addressBookId)
-	{
+	public function deleteAddressBook($addressBookId) {
 		$this->logger->trace("addressBookId: %s", $addressBookId);
 		$success = $this->gDavBackend->DeleteFolder($addressBookId);
 		// TODO evaluate $success
@@ -122,8 +117,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function getCards($addressbookId)
-	{
+	public function getCards($addressbookId) {
 		$result = $this->gDavBackend->GetObjects($addressbookId, static::FILE_EXTENSION);
 		$this->logger->trace("addressbookId: %s found %d objects", $addressbookId, count($result));
 
@@ -144,8 +138,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function getCard($addressBookId, $cardUri, $mapifolder = null)
-	{
+	public function getCard($addressBookId, $cardUri, $mapifolder = null) {
 		$this->logger->trace("addressBookId: %s - cardUri: %s", $addressBookId, $cardUri);
 
 		if (!$mapifolder) {
@@ -208,8 +201,7 @@ 
 	 *
 	 * @return null|string
 	 */
-	public function createCard($addressBookId, $cardUri, $cardData)
-	{
+	public function createCard($addressBookId, $cardUri, $cardData) {
 		$this->logger->trace("addressBookId: %s - cardUri: %s - cardData: %s", $addressBookId, $cardUri, $cardData);
 		$objectId = $this->gDavBackend->GetObjectIdFromObjectUri($cardUri, static::FILE_EXTENSION);
 		$folder = $this->gDavBackend->GetMapiFolder($addressBookId);
@@ -248,8 +240,7 @@ 
 	 *
 	 * @return null|string
 	 */
-	public function updateCard($addressBookId, $cardUri, $cardData)
-	{
+	public function updateCard($addressBookId, $cardUri, $cardData) {
 		$this->logger->trace("addressBookId: %s - cardUri: %s - cardData: %s", $addressBookId, $cardUri, $cardData);
 
 		$mapimessage = $this->gDavBackend->GetMapiMessageForId($addressBookId, $cardUri, null, static::FILE_EXTENSION);
@@ -270,8 +261,7 @@ 
 	 *
 	 * @return null|string
 	 */
-	private function setData($addressBookId, $mapimessage, $vcf)
-	{
+	private function setData($addressBookId, $mapimessage, $vcf) {
 		$this->logger->trace("mapimessage: %s - vcf: %s", $mapimessage, $vcf);
 		$store = $this->gDavBackend->GetStoreById($addressBookId);
 		$session = $this->gDavBackend->GetSession();
@@ -296,8 +286,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function deleteCard($addressBookId, $cardUri)
-	{
+	public function deleteCard($addressBookId, $cardUri) {
 		$this->logger->trace("addressBookId: %s - cardUri: %s", $addressBookId, $cardUri);
 		$mapifolder = $this->gDavBackend->GetMapiFolder($addressBookId);
 		$objectId = $this->gDavBackend->GetObjectIdFromObjectUri($cardUri, static::FILE_EXTENSION);
@@ -366,8 +355,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function getChangesForAddressBook($addressBookId, $syncToken, $syncLevel, $limit = null)
-	{
+	public function getChangesForAddressBook($addressBookId, $syncToken, $syncLevel, $limit = null) {
 		$this->logger->trace("addressBookId: %s - syncToken: %s - syncLevel: %d - limit: %d", $addressBookId, $syncToken, $syncLevel, $limit);
 
 		return $this->gDavBackend->Sync($addressBookId, $syncToken, static::FILE_EXTENSION, $limit);

tests/grommunio-dav/GrommunioCardDavBackendTest.php

Braces +3 added lines, -6 removed lines

@@ -25,8 +25,7 @@ 
 	 *
 	 * @see PHPUnit_Framework_TestCase::setUp()
 	 */
-	protected function setUp()
-	{
+	protected function setUp() {
 		$gloggerMock = $this->getMockBuilder(GLogger::class)->disableOriginalConstructor()->getMock();
 		$this->gDavBackendMock = $this->getMockBuilder(GrommunioDavBackend::class)->disableOriginalConstructor()->getMock();
 		$this->kCardDavBackend = new GrommunioCardDavBackend($this->gDavBackendMock, $gloggerMock);
@@ -37,8 +36,7 @@ 
 	 *
 	 * @see PHPUnit_Framework_TestCase::tearDown()
 	 */
-	protected function tearDown()
-	{
+	protected function tearDown() {
 		$this->kCardDavBackend = null;
 		$this->gDavBackendMock = null;
 	}
@@ -46,8 +44,7 @@ 
 	/**
 	 * Tests if the constructor is created without errors.
 	 */
-	public function testConstruct()
-	{
+	public function testConstruct() {
 		$this->assertTrue(is_object($this->kCardDavBackend));
 	}
 }

tests/grommunio-dav/GrommunioDavBackendTest.php

Spacing +4 added lines, -4 removed lines

@@ -69,10 +69,10 @@
 	public function ObjectUriProvider()
 	{
 		return [
-			['1234.ics', '.ics', '1234'],               // ok, cut .ics
-			['5678AF.vcf', '.vcf', '5678AF'],           // ok, cut .vcf
-			['123400.vcf', '.ics', '123400.vcf'],       // different extension, return as is
-			['1234.ics', '.vcf', '1234.ics'],            // different extension, return as is
+			['1234.ics', '.ics', '1234'], // ok, cut .ics
+			['5678AF.vcf', '.vcf', '5678AF'], // ok, cut .vcf
+			['123400.vcf', '.ics', '123400.vcf'], // different extension, return as is
+			['1234.ics', '.vcf', '1234.ics'], // different extension, return as is
 		];
 	}
 }

Braces +5 added lines, -10 removed lines

@@ -23,8 +23,7 @@ 
 	 *
 	 * @see PHPUnit_Framework_TestCase::setUp()
 	 */
-	protected function setUp()
-	{
+	protected function setUp() {
 		$gloggerMock = $this->getMockBuilder(GLogger::class)->disableOriginalConstructor()->getMock();
 		$this->gDavBackend = new GrommunioDavBackend($gloggerMock);
 	}
@@ -34,16 +33,14 @@ 
 	 *
 	 * @see PHPUnit_Framework_TestCase::tearDown()
 	 */
-	protected function tearDown()
-	{
+	protected function tearDown() {
 		$this->gDavBackend = null;
 	}
 
 	/**
 	 * Tests if the constructor is created without errors.
 	 */
-	public function testConstruct()
-	{
+	public function testConstruct() {
 		$this->assertTrue(is_object($this->gDavBackend));
 	}
 
@@ -56,8 +53,7 @@ 
 	 *
 	 * @dataProvider ObjectUriProvider
 	 */
-	public function testGetObjectIdFromObjectUri($objectUri, $extension, $expected)
-	{
+	public function testGetObjectIdFromObjectUri($objectUri, $extension, $expected) {
 		$this->assertEquals($expected, $this->gDavBackend->GetObjectIdFromObjectUri($objectUri, $extension));
 	}
 
@@ -66,8 +62,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function ObjectUriProvider()
-	{
+	public function ObjectUriProvider() {
 		return [
 			['1234.ics', '.ics', '1234'],               // ok, cut .ics
 			['5678AF.vcf', '.vcf', '5678AF'],           // ok, cut .vcf