Passed
Push — master ( c00d6f...a085a8 )
by Morris
09:54 queued 11s
created
apps/dav/composer/composer/autoload_psr4.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,5 +6,5 @@
 block discarded – undo
6 6
 $baseDir = $vendorDir;
7 7
 
8 8
 return array(
9
-    'OCA\\DAV\\' => array($baseDir . '/../lib'),
9
+    'OCA\\DAV\\' => array($baseDir.'/../lib'),
10 10
 );
Please login to merge, or discard this patch.
apps/comments/composer/composer/autoload_psr4.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,5 +6,5 @@
 block discarded – undo
6 6
 $baseDir = $vendorDir;
7 7
 
8 8
 return array(
9
-    'OCA\\Comments\\' => array($baseDir . '/../lib'),
9
+    'OCA\\Comments\\' => array($baseDir.'/../lib'),
10 10
 );
Please login to merge, or discard this patch.
lib/private/Remote/Instance.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	 * @return string The of of the remote server with protocol
66 66
 	 */
67 67
 	public function getFullUrl() {
68
-		return $this->getProtocol() . '://' . $this->getUrl();
68
+		return $this->getProtocol().'://'.$this->getUrl();
69 69
 	}
70 70
 
71 71
 	/**
@@ -103,17 +103,17 @@  discard block
 block discarded – undo
103 103
 		if ($this->status) {
104 104
 			return $this->status;
105 105
 		}
106
-		$key = 'remote/' . $this->url . '/status';
107
-		$httpsKey = 'remote/' . $this->url . '/https';
106
+		$key = 'remote/'.$this->url.'/status';
107
+		$httpsKey = 'remote/'.$this->url.'/https';
108 108
 		$status = $this->cache->get($key);
109 109
 		if (!$status) {
110
-			$response = $this->downloadStatus('https://' . $this->getUrl() . '/status.php');
110
+			$response = $this->downloadStatus('https://'.$this->getUrl().'/status.php');
111 111
 			$protocol = 'https';
112 112
 			if (!$response) {
113 113
 				if ($status = $this->cache->get($httpsKey)) {
114
-					throw new \Exception('refusing to connect to remote instance(' . $this->url . ') over http that was previously accessible over https');
114
+					throw new \Exception('refusing to connect to remote instance('.$this->url.') over http that was previously accessible over https');
115 115
 				}
116
-				$response = $this->downloadStatus('http://' . $this->getUrl() . '/status.php');
116
+				$response = $this->downloadStatus('http://'.$this->getUrl().'/status.php');
117 117
 				$protocol = 'http';
118 118
 			} else {
119 119
 				$this->cache->set($httpsKey, true, 60 * 60 * 24 * 365);
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 				$this->cache->set($key, $status, 5 * 60);
127 127
 				$this->status = $status;
128 128
 			} else {
129
-				throw new NotFoundException('Remote server not found at address ' . $this->url);
129
+				throw new NotFoundException('Remote server not found at address '.$this->url);
130 130
 			}
131 131
 		}
132 132
 		return $status;
Please login to merge, or discard this patch.
lib/private/Remote/Api/ApiBase.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 	 * @throws \InvalidArgumentException
61 61
 	 */
62 62
 	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
63
-		$fullUrl = trim($this->instance->getFullUrl(), '/') . '/' . $url;
63
+		$fullUrl = trim($this->instance->getFullUrl(), '/').'/'.$url;
64 64
 		$options = [
65 65
 			'query' => $query,
66 66
 			'headers' => $this->addDefaultHeaders($headers),
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 				$response = $client->options($fullUrl, $options);
90 90
 				break;
91 91
 			default:
92
-				throw new \InvalidArgumentException('Invalid method ' . $method);
92
+				throw new \InvalidArgumentException('Invalid method '.$method);
93 93
 		}
94 94
 
95 95
 		return $response->getBody();
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/BirthdayCalendar/EnablePlugin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
 	 *
79 79
 	 * @return string
80 80
 	 */
81
-	public function getPluginName()	{
81
+	public function getPluginName() {
82 82
 		return 'nc-enable-birthday-calendar';
83 83
 	}
84 84
 
Please login to merge, or discard this patch.
lib/private/Remote/Api/OCS.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
 	 */
44 44
 	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
45 45
 		try {
46
-			$response = json_decode(parent::request($method, 'ocs/v2.php/' . $url, $body, $query, $headers), true);
46
+			$response = json_decode(parent::request($method, 'ocs/v2.php/'.$url, $body, $query, $headers), true);
47 47
 		} catch (ClientException $e) {
48 48
 			if ($e->getResponse()->getStatusCode() === 404) {
49 49
 				throw new NotFoundException();
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 			throw new NotFoundException();
64 64
 		}
65 65
 		if ($response['ocs']['meta']['status'] !== 'ok') {
66
-			throw new \Exception('Unknown ocs error ' . $response['ocs']['meta']['message']);
66
+			throw new \Exception('Unknown ocs error '.$response['ocs']['meta']['message']);
67 67
 		}
68 68
 
69 69
 		return $response['ocs']['data'];
@@ -78,13 +78,13 @@  discard block
 block discarded – undo
78 78
 	private function checkResponseArray(array $data, $type, array $keys) {
79 79
 		foreach ($keys as $key) {
80 80
 			if (!array_key_exists($key, $data)) {
81
-				throw new \Exception('Invalid ' . $type . ' response, expected field ' . $key . ' not found');
81
+				throw new \Exception('Invalid '.$type.' response, expected field '.$key.' not found');
82 82
 			}
83 83
 		}
84 84
 	}
85 85
 
86 86
 	public function getUser($userId) {
87
-		$result = $this->request('get', 'cloud/users/' . $userId);
87
+		$result = $this->request('get', 'cloud/users/'.$userId);
88 88
 		$this->checkResponseArray($result, 'user', User::EXPECTED_KEYS);
89 89
 		return new User($result);
90 90
 	}
Please login to merge, or discard this patch.
apps/dav/lib/Command/SyncBirthdayCalendar.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -109,7 +109,7 @@
 block discarded – undo
109 109
 		$output->writeln('');
110 110
 	}
111 111
 
112
-	protected function verifyEnabled () {
112
+	protected function verifyEnabled() {
113 113
 		$isEnabled = $this->config->getAppValue('dav', 'generateBirthdayCalendar', 'yes');
114 114
 
115 115
 		if ($isEnabled !== 'yes') {
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/CalendarObject.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -72,19 +72,19 @@  discard block
 block discarded – undo
72 72
 	private static function createConfidentialObject(Component\VCalendar $vObject) {
73 73
 		/** @var Component $vElement */
74 74
 		$vElement = null;
75
-		if(isset($vObject->VEVENT)) {
75
+		if (isset($vObject->VEVENT)) {
76 76
 			$vElement = $vObject->VEVENT;
77 77
 		}
78
-		if(isset($vObject->VJOURNAL)) {
78
+		if (isset($vObject->VJOURNAL)) {
79 79
 			$vElement = $vObject->VJOURNAL;
80 80
 		}
81
-		if(isset($vObject->VTODO)) {
81
+		if (isset($vObject->VTODO)) {
82 82
 			$vElement = $vObject->VTODO;
83 83
 		}
84
-		if(!is_null($vElement)) {
84
+		if (!is_null($vElement)) {
85 85
 			foreach ($vElement->children() as &$property) {
86 86
 				/** @var Property $property */
87
-				switch($property->name) {
87
+				switch ($property->name) {
88 88
 					case 'CREATED':
89 89
 					case 'DTSTART':
90 90
 					case 'RRULE':
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 	private function removeVAlarms(Component\VCalendar $vObject) {
113 113
 		$subcomponents = $vObject->getComponents();
114 114
 
115
-		foreach($subcomponents as $subcomponent) {
115
+		foreach ($subcomponents as $subcomponent) {
116 116
 			unset($subcomponent->VALARM);
117 117
 		}
118 118
 	}
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/Activity/Provider/Todo.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -46,26 +46,26 @@  discard block
 block discarded – undo
46 46
 			$event->setIcon($this->url->getAbsoluteURL($this->url->imagePath('core', 'actions/checkmark.svg')));
47 47
 		}
48 48
 
49
-		if ($event->getSubject() === self::SUBJECT_OBJECT_ADD . '_todo') {
49
+		if ($event->getSubject() === self::SUBJECT_OBJECT_ADD.'_todo') {
50 50
 			$subject = $this->l->t('{actor} created todo {todo} in list {calendar}');
51
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_ADD . '_todo_self') {
51
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_ADD.'_todo_self') {
52 52
 			$subject = $this->l->t('You created todo {todo} in list {calendar}');
53
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_DELETE . '_todo') {
53
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_DELETE.'_todo') {
54 54
 			$subject = $this->l->t('{actor} deleted todo {todo} from list {calendar}');
55
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_DELETE . '_todo_self') {
55
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_DELETE.'_todo_self') {
56 56
 			$subject = $this->l->t('You deleted todo {todo} from list {calendar}');
57
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE . '_todo') {
57
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE.'_todo') {
58 58
 			$subject = $this->l->t('{actor} updated todo {todo} in list {calendar}');
59
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE . '_todo_self') {
59
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE.'_todo_self') {
60 60
 			$subject = $this->l->t('You updated todo {todo} in list {calendar}');
61 61
 
62
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE . '_todo_completed') {
62
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE.'_todo_completed') {
63 63
 			$subject = $this->l->t('{actor} solved todo {todo} in list {calendar}');
64
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE . '_todo_completed_self') {
64
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE.'_todo_completed_self') {
65 65
 			$subject = $this->l->t('You solved todo {todo} in list {calendar}');
66
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE . '_todo_needs_action') {
66
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE.'_todo_needs_action') {
67 67
 			$subject = $this->l->t('{actor} reopened todo {todo} in list {calendar}');
68
-		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE . '_todo_needs_action_self') {
68
+		} else if ($event->getSubject() === self::SUBJECT_OBJECT_UPDATE.'_todo_needs_action_self') {
69 69
 			$subject = $this->l->t('You reopened todo {todo} in list {calendar}');
70 70
 		} else {
71 71
 			throw new \InvalidArgumentException();
@@ -90,21 +90,21 @@  discard block
 block discarded – undo
90 90
 		// Nextcloud 13+
91 91
 		if (isset($parameters['calendar'])) {
92 92
 			switch ($subject) {
93
-				case self::SUBJECT_OBJECT_ADD . '_todo':
94
-				case self::SUBJECT_OBJECT_DELETE . '_todo':
95
-				case self::SUBJECT_OBJECT_UPDATE . '_todo':
96
-				case self::SUBJECT_OBJECT_UPDATE . '_todo_completed':
97
-				case self::SUBJECT_OBJECT_UPDATE . '_todo_needs_action':
93
+				case self::SUBJECT_OBJECT_ADD.'_todo':
94
+				case self::SUBJECT_OBJECT_DELETE.'_todo':
95
+				case self::SUBJECT_OBJECT_UPDATE.'_todo':
96
+				case self::SUBJECT_OBJECT_UPDATE.'_todo_completed':
97
+				case self::SUBJECT_OBJECT_UPDATE.'_todo_needs_action':
98 98
 					return [
99 99
 						'actor' => $this->generateUserParameter($parameters['actor']),
100 100
 						'calendar' => $this->generateCalendarParameter($parameters['calendar'], $this->l),
101 101
 						'todo' => $this->generateObjectParameter($parameters['object']),
102 102
 					];
103
-				case self::SUBJECT_OBJECT_ADD . '_todo_self':
104
-				case self::SUBJECT_OBJECT_DELETE . '_todo_self':
105
-				case self::SUBJECT_OBJECT_UPDATE . '_todo_self':
106
-				case self::SUBJECT_OBJECT_UPDATE . '_todo_completed_self':
107
-				case self::SUBJECT_OBJECT_UPDATE . '_todo_needs_action_self':
103
+				case self::SUBJECT_OBJECT_ADD.'_todo_self':
104
+				case self::SUBJECT_OBJECT_DELETE.'_todo_self':
105
+				case self::SUBJECT_OBJECT_UPDATE.'_todo_self':
106
+				case self::SUBJECT_OBJECT_UPDATE.'_todo_completed_self':
107
+				case self::SUBJECT_OBJECT_UPDATE.'_todo_needs_action_self':
108 108
 					return [
109 109
 						'calendar' => $this->generateCalendarParameter($parameters['calendar'], $this->l),
110 110
 						'todo' => $this->generateObjectParameter($parameters['object']),
@@ -118,23 +118,23 @@  discard block
 block discarded – undo
118 118
 		// Otherwise if people upgrade over multiple releases in a short period,
119 119
 		// they will get the dead entries in their stream.
120 120
 		switch ($subject) {
121
-			case self::SUBJECT_OBJECT_ADD . '_todo':
122
-			case self::SUBJECT_OBJECT_DELETE . '_todo':
123
-			case self::SUBJECT_OBJECT_UPDATE . '_todo':
124
-			case self::SUBJECT_OBJECT_UPDATE . '_todo_completed':
125
-			case self::SUBJECT_OBJECT_UPDATE . '_todo_needs_action':
121
+			case self::SUBJECT_OBJECT_ADD.'_todo':
122
+			case self::SUBJECT_OBJECT_DELETE.'_todo':
123
+			case self::SUBJECT_OBJECT_UPDATE.'_todo':
124
+			case self::SUBJECT_OBJECT_UPDATE.'_todo_completed':
125
+			case self::SUBJECT_OBJECT_UPDATE.'_todo_needs_action':
126 126
 				return [
127 127
 					'actor' => $this->generateUserParameter($parameters[0]),
128
-					'calendar' => $this->generateLegacyCalendarParameter((int)$event->getObjectId(), $parameters[1]),
128
+					'calendar' => $this->generateLegacyCalendarParameter((int) $event->getObjectId(), $parameters[1]),
129 129
 					'todo' => $this->generateObjectParameter($parameters[2]),
130 130
 				];
131
-			case self::SUBJECT_OBJECT_ADD . '_todo_self':
132
-			case self::SUBJECT_OBJECT_DELETE . '_todo_self':
133
-			case self::SUBJECT_OBJECT_UPDATE . '_todo_self':
134
-			case self::SUBJECT_OBJECT_UPDATE . '_todo_completed_self':
135
-			case self::SUBJECT_OBJECT_UPDATE . '_todo_needs_action_self':
131
+			case self::SUBJECT_OBJECT_ADD.'_todo_self':
132
+			case self::SUBJECT_OBJECT_DELETE.'_todo_self':
133
+			case self::SUBJECT_OBJECT_UPDATE.'_todo_self':
134
+			case self::SUBJECT_OBJECT_UPDATE.'_todo_completed_self':
135
+			case self::SUBJECT_OBJECT_UPDATE.'_todo_needs_action_self':
136 136
 				return [
137
-					'calendar' => $this->generateLegacyCalendarParameter((int)$event->getObjectId(), $parameters[1]),
137
+					'calendar' => $this->generateLegacyCalendarParameter((int) $event->getObjectId(), $parameters[1]),
138 138
 					'todo' => $this->generateObjectParameter($parameters[2]),
139 139
 				];
140 140
 		}
Please login to merge, or discard this patch.