Add visibility to all methods and position of static keyword

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

7 files changed, 24 insertions, 24 deletions

diff --git a/apps/dav/lib/DAV/GroupPrincipalBackend.php b/apps/dav/lib/DAV/GroupPrincipalBackend.php
index f03c7cc2bdd..cfa896c4f2c 100644
--- a/apps/dav/lib/DAV/GroupPrincipalBackend.php
+++ b/apps/dav/lib/DAV/GroupPrincipalBackend.php
@@ -167,7 +167,7 @@ class GroupPrincipalBackend implements BackendInterface {
 	 * @param PropPatch $propPatch
 	 * @return int
 	 */
-	function updatePrincipal($path, PropPatch $propPatch) {
+	public function updatePrincipal($path, PropPatch $propPatch) {
 		return 0;
 	}
 
@@ -177,7 +177,7 @@ class GroupPrincipalBackend implements BackendInterface {
 	 * @param string $test
 	 * @return array
 	 */
-	function searchPrincipals($prefixPath, array $searchProperties, $test = 'allof') {
+	public function searchPrincipals($prefixPath, array $searchProperties, $test = 'allof') {
 		$results = [];
 
 		if (\count($searchProperties) === 0) {
@@ -257,7 +257,7 @@ class GroupPrincipalBackend implements BackendInterface {
 	 * @param string $principalPrefix
 	 * @return string
 	 */
-	function findByUri($uri, $principalPrefix) {
+	public function findByUri($uri, $principalPrefix) {
 		// If sharing is disabled, return the empty array
 		$shareAPIEnabled = $this->shareManager->shareApiEnabled();
 		if (!$shareAPIEnabled) {
diff --git a/apps/dav/lib/DAV/PublicAuth.php b/apps/dav/lib/DAV/PublicAuth.php
index 407f1a2853f..d37c2dd46f6 100644
--- a/apps/dav/lib/DAV/PublicAuth.php
+++ b/apps/dav/lib/DAV/PublicAuth.php
@@ -67,7 +67,7 @@ class PublicAuth implements BackendInterface {
 	 * @param ResponseInterface $response
 	 * @return array
 	 */
-	function check(RequestInterface $request, ResponseInterface $response) {
+	public function check(RequestInterface $request, ResponseInterface $response) {
 		if ($this->isRequestPublic($request)) {
 			return [true, "principals/system/public"];
 		}
@@ -77,7 +77,7 @@ class PublicAuth implements BackendInterface {
 	/**
 	 * @inheritdoc
 	 */
-	function challenge(RequestInterface $request, ResponseInterface $response) {
+	public function challenge(RequestInterface $request, ResponseInterface $response) {
 	}
 
 	/**
diff --git a/apps/dav/lib/DAV/Sharing/IShareable.php b/apps/dav/lib/DAV/Sharing/IShareable.php
index 1293721040a..ffe9e3691dc 100644
--- a/apps/dav/lib/DAV/Sharing/IShareable.php
+++ b/apps/dav/lib/DAV/Sharing/IShareable.php
@@ -49,7 +49,7 @@ interface IShareable extends INode {
 	 * @param array $remove
 	 * @return void
 	 */
-	function updateShares(array $add, array $remove);
+	public function updateShares(array $add, array $remove);
 
 	/**
 	 * Returns the list of people whom this resource is shared with.
@@ -63,7 +63,7 @@ interface IShareable extends INode {
 	 *
 	 * @return array
 	 */
-	function getShares();
+	public function getShares();
 
 	/**
 	 * @return int
diff --git a/apps/dav/lib/DAV/Sharing/Plugin.php b/apps/dav/lib/DAV/Sharing/Plugin.php
index 67eed9c9b4a..148afa323dd 100644
--- a/apps/dav/lib/DAV/Sharing/Plugin.php
+++ b/apps/dav/lib/DAV/Sharing/Plugin.php
@@ -72,7 +72,7 @@ class Plugin extends ServerPlugin {
 	 *
 	 * @return string[]
 	 */
-	function getFeatures() {
+	public function getFeatures() {
 		return ['oc-resource-sharing'];
 	}
 
@@ -84,7 +84,7 @@ class Plugin extends ServerPlugin {
 	 *
 	 * @return string
 	 */
-	function getPluginName() {
+	public function getPluginName() {
 		return 'oc-resource-sharing';
 	}
 
@@ -99,7 +99,7 @@ class Plugin extends ServerPlugin {
 	 * @param Server $server
 	 * @return void
 	 */
-	function initialize(Server $server) {
+	public function initialize(Server $server) {
 		$this->server = $server;
 		$this->server->xml->elementMap['{' . Plugin::NS_OWNCLOUD . '}share'] = ShareRequest::class;
 		$this->server->xml->elementMap['{' . Plugin::NS_OWNCLOUD . '}invite'] = Invite::class;
@@ -115,7 +115,7 @@ class Plugin extends ServerPlugin {
 	 * @param ResponseInterface $response
 	 * @return null|false
 	 */
-	function httpPost(RequestInterface $request, ResponseInterface $response) {
+	public function httpPost(RequestInterface $request, ResponseInterface $response) {
 		$path = $request->getPath();
 
 		// Only handling xml
@@ -188,7 +188,7 @@ class Plugin extends ServerPlugin {
 	 * @param INode $node
 	 * @return void
 	 */
-	function propFind(PropFind $propFind, INode $node) {
+	public function propFind(PropFind $propFind, INode $node) {
 		if ($node instanceof IShareable) {
 			$propFind->handle('{' . Plugin::NS_OWNCLOUD . '}invite', function () use ($node) {
 				return new Invite(
diff --git a/apps/dav/lib/DAV/Sharing/Xml/Invite.php b/apps/dav/lib/DAV/Sharing/Xml/Invite.php
index 68aab171ab7..315ede2ce9c 100644
--- a/apps/dav/lib/DAV/Sharing/Xml/Invite.php
+++ b/apps/dav/lib/DAV/Sharing/Xml/Invite.php
@@ -84,7 +84,7 @@ class Invite implements XmlSerializable {
 	 *
 	 * @param array $users
 	 */
-	function __construct(array $users, array $organizer = null) {
+	public function __construct(array $users, array $organizer = null) {
 		$this->users = $users;
 		$this->organizer = $organizer;
 	}
@@ -94,7 +94,7 @@ class Invite implements XmlSerializable {
 	 *
 	 * @return array
 	 */
-	function getValue() {
+	public function getValue() {
 		return $this->users;
 	}
 
@@ -117,7 +117,7 @@ class Invite implements XmlSerializable {
 	 * @param Writer $writer
 	 * @return void
 	 */
-	function xmlSerialize(Writer $writer) {
+	public function xmlSerialize(Writer $writer) {
 		$cs = '{' . Plugin::NS_OWNCLOUD . '}';
 
 		if (!is_null($this->organizer)) {
diff --git a/apps/dav/lib/DAV/Sharing/Xml/ShareRequest.php b/apps/dav/lib/DAV/Sharing/Xml/ShareRequest.php
index d76e65aa232..c973908f5a8 100644
--- a/apps/dav/lib/DAV/Sharing/Xml/ShareRequest.php
+++ b/apps/dav/lib/DAV/Sharing/Xml/ShareRequest.php
@@ -38,12 +38,12 @@ class ShareRequest implements XmlDeserializable {
 	 * @param array $set
 	 * @param array $remove
 	 */
-	function __construct(array $set, array $remove) {
+	public function __construct(array $set, array $remove) {
 		$this->set = $set;
 		$this->remove = $remove;
 	}
 
-	static function xmlDeserialize(Reader $reader) {
+	public static function xmlDeserialize(Reader $reader) {
 		$elements = $reader->parseInnerTree([
 			'{' . Plugin::NS_OWNCLOUD. '}set'    => 'Sabre\\Xml\\Element\\KeyValue',
 			'{' . Plugin::NS_OWNCLOUD . '}remove' => 'Sabre\\Xml\\Element\\KeyValue',
diff --git a/apps/dav/lib/DAV/SystemPrincipalBackend.php b/apps/dav/lib/DAV/SystemPrincipalBackend.php
index c369f4da375..ebe1d61db4a 100644
--- a/apps/dav/lib/DAV/SystemPrincipalBackend.php
+++ b/apps/dav/lib/DAV/SystemPrincipalBackend.php
@@ -43,7 +43,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param string $prefixPath
 	 * @return array
 	 */
-	function getPrincipalsByPrefix($prefixPath) {
+	public function getPrincipalsByPrefix($prefixPath) {
 		$principals = [];
 
 		if ($prefixPath === 'principals/system') {
@@ -68,7 +68,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param string $path
 	 * @return array
 	 */
-	function getPrincipalByPath($path) {
+	public function getPrincipalByPath($path) {
 		if ($path === 'principals/system/system') {
 			$principal = [
 				'uri' => 'principals/system/system',
@@ -103,7 +103,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param \Sabre\DAV\PropPatch $propPatch
 	 * @return void
 	 */
-	function updatePrincipal($path, \Sabre\DAV\PropPatch $propPatch) {
+	public function updatePrincipal($path, \Sabre\DAV\PropPatch $propPatch) {
 	}
 
 	/**
@@ -135,7 +135,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param string $test
 	 * @return array
 	 */
-	function searchPrincipals($prefixPath, array $searchProperties, $test = 'allof') {
+	public function searchPrincipals($prefixPath, array $searchProperties, $test = 'allof') {
 		return [];
 	}
 
@@ -145,7 +145,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param string $principal
 	 * @return array
 	 */
-	function getGroupMemberSet($principal) {
+	public function getGroupMemberSet($principal) {
 		// TODO: for now the group principal has only one member, the user itself
 		$principal = $this->getPrincipalByPath($principal);
 		if (!$principal) {
@@ -161,7 +161,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param string $principal
 	 * @return array
 	 */
-	function getGroupMembership($principal) {
+	public function getGroupMembership($principal) {
 		list($prefix, ) = \Sabre\Uri\split($principal);
 
 		if ($prefix === 'principals/system') {
@@ -184,7 +184,7 @@ class SystemPrincipalBackend extends AbstractBackend {
 	 * @param array $members
 	 * @return void
 	 */
-	function setGroupMemberSet($principal, array $members) {
+	public function setGroupMemberSet($principal, array $members) {
 		throw new \Sabre\DAV\Exception('Setting members of the group is not supported yet');
 	}
 }