owncloud / core

lib/private/preview.php

Doc Comments +11 added lines, -11 removed lines

@@ -201,7 +201,7 @@ 
 	/**
 	 * returns the max width set in ownCloud's config
 	 *
-	 * @return string
+	 * @return integer
 	 */
 	public function getConfigMaxX() {
 		return $this->configMaxWidth;
@@ -210,7 +210,7 @@ 
 	/**
 	 * returns the max height set in ownCloud's config
 	 *
-	 * @return string
+	 * @return integer
 	 */
 	public function getConfigMaxY() {
 		return $this->configMaxHeight;
@@ -545,7 +545,7 @@ 
 	/**
 	 * Determines the size of the preview we should be looking for in the cache
 	 *
-	 * @return int[]
+	 * @return integer[]
 	 */
 	private function simulatePreviewDimensions() {
 		$askedWidth = $this->getMaxX();
@@ -569,7 +569,7 @@ 
 	 *
 	 * @param int $originalWidth
 	 * @param int $originalHeight
-	 * @return \int[]
+	 * @return integer[]
 	 */
 	private function applyAspectRatio($askedWidth, $askedHeight, $originalWidth = 0, $originalHeight = 0) {
 		if(!$originalWidth){
@@ -601,7 +601,7 @@ 
 	 * @param int $askedHeight
 	 * @param int $previewWidth
 	 * @param int $previewHeight
-	 * @return \int[]
+	 * @return integer[]
 	 */
 	private function applyCover($askedWidth, $askedHeight, $previewWidth, $previewHeight) {
 		$originalRatio = $previewWidth / $previewHeight;
@@ -627,7 +627,7 @@ 
 	 * @param int $askedWidth
 	 * @param int $askedHeight
 	 *
-	 * @return \int[]
+	 * @return integer[]
 	 */
 	private function fixSize($askedWidth, $askedHeight) {
 		if ($this->scalingUp) {
@@ -920,9 +920,9 @@ 
 	 * @param int $askedWidth
 	 * @param int $askedHeight
 	 * @param int $previewWidth
-	 * @param null $previewHeight
+	 * @param integer $previewHeight
 	 *
-	 * @return int[]
+	 * @return double[]
 	 */
 	private function scale($image, $askedWidth, $askedHeight, $previewWidth, $previewHeight) {
 		$scalingUp = $this->getScalingUp();
@@ -970,7 +970,7 @@ 
 	 * @param int $askedWidth
 	 * @param int $askedHeight
 	 * @param int $previewWidth
-	 * @param null $previewHeight
+	 * @param double $previewHeight
 	 */
 	private function crop($image, $askedWidth, $askedHeight, $previewWidth, $previewHeight = null) {
 		$cropX = floor(abs($askedWidth - $previewWidth) * 0.5);
@@ -989,7 +989,7 @@ 
 	 * @param int $askedWidth
 	 * @param int $askedHeight
 	 * @param int $previewWidth
-	 * @param null $previewHeight
+	 * @param double $previewHeight
 	 */
 	private function cropAndFill($image, $askedWidth, $askedHeight, $previewWidth, $previewHeight) {
 		if ($previewWidth > $askedWidth) {
@@ -1217,7 +1217,7 @@ 
 	 * @param int $maxDim
 	 * @param string $dimName
 	 *
-	 * @return mixed
+	 * @return integer
 	 */
 	private function limitMaxDim($dim, $maxDim, $dimName) {
 		if (!is_null($maxDim)) {

lib/private/repair.php

Doc Comments +3 added lines, -1 removed lines

@@ -123,7 +123,7 @@ 
 	 * Returns expensive repair steps to be run on the
 	 * command line with a special option.
 	 *
-	 * @return array of RepairStep instances
+	 * @return OldGroupMembershipShares[] of RepairStep instances
 	 */
 	public static function getExpensiveRepairSteps() {
 		return [
@@ -161,6 +161,8 @@ 
 	 * {@inheritDoc}
 	 *
 	 * Re-declared as public to allow invocation from within the closure above in php 5.3
+	 * @param string $scope
+	 * @param string $method
 	 */
 	public function emit($scope, $method, array $arguments = array()) {
 		parent::emit($scope, $method, $arguments);

lib/private/server.php

Doc Comments +1 added lines, -1 removed lines

@@ -856,7 +856,7 @@
 	 * Get the certificate manager for the user
 	 *
 	 * @param string $userId (optional) if not specified the current loggedin user is used
-	 * @return \OCP\ICertificateManager | null if $uid is null and no user is logged in
+	 * @return null|CertificateManager | null if $uid is null and no user is logged in
 	 */
 	public function getCertificateManager($userId = null) {
 		if (is_null($userId)) {

lib/private/session/cryptosessiondata.php

Doc Comments +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
 
 	/**
 	 * @param mixed $offset
-	 * @return mixed
+	 * @return string|null
 	 */
 	public function offsetGet($offset) {
 		return $this->get($offset);

lib/private/setup/abstractdatabase.php

Doc Comments +3 added lines

@@ -91,5 +91,8 @@
 		$this->tablePrefix = $dbTablePrefix;
 	}
 
+	/**
+	 * @param string $userName
+	 */
 	abstract public function setupDatabase($userName);
 }

lib/private/setup/mysql.php

Doc Comments +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
 	}
 
 	/**
-	 * @param IDbConnection $connection
+	 * @param IDBConnection $connection
 	 * @throws \OC\DatabaseSetupException
 	 */
 	private function createDBUser($connection) {

lib/private/share/helper.php

Doc Comments +2 added lines, -2 removed lines

@@ -227,7 +227,7 @@ 
 	 *
 	 * all return: http://localhost
 	 *
-	 * @param string $shareWith
+	 * @param string $remote
 	 * @return string
 	 */
 	protected static function fixRemoteURL($remote) {
@@ -244,7 +244,7 @@ 
 	 * split user and remote from federated cloud id
 	 *
 	 * @param string $id
-	 * @return array
+	 * @return string[]
 	 * @throws HintException
 	 */
 	public static function splitUserRemote($id) {

lib/private/share/share.php

Doc Comments +10 added lines

@@ -2211,6 +2211,16 @@
 		return $id ? $id : false;
 	}
 
+	/**
+	 * @param string $itemType
+	 * @param string $itemSource
+	 * @param integer $shareType
+	 * @param string $shareWith
+	 * @param string $uidOwner
+	 * @param integer $permissions
+	 * @param string|null $itemSourceName
+	 * @param null|\DateTime $expirationDate
+	 */
 	private static function checkReshare($itemType, $itemSource, $shareType, $shareWith, $uidOwner, $permissions, $itemSourceName, $expirationDate) {
 		$backend = self::getBackend($itemType);
 

lib/private/share20/manager.php

Unused Use Statements -1 removed lines

@@ -27,7 +27,6 @@
 use OCP\IUser;
 use OCP\ILogger;
 use OCP\Files\Folder;
-
 use OC\Share20\Exception\ShareNotFound;
 
 /**

Doc Comments +1 added lines, -1 removed lines

@@ -201,7 +201,7 @@
 	 * Validate if the expiration date fits the system settings
 	 *
 	 * @param \DateTime $expireDate The current expiration date (can be null)
-	 * @return \DateTime|null The expiration date or null if $expireDate was null and it is not required
+	 * @return \DateTime The expiration date or null if $expireDate was null and it is not required
 	 * @throws \OC\HintException
 	 */
 	protected function validateExpiredate($expireDate) {