WebSVN – oidplus – Diff – /trunk/plugins/viathinksoft/publicPages/095_attachments/OIDplusPagePublicAttachments.class.php

 \defined('INSIDE_OIDPLUS') or die;
 // phpcs:enable PSR1.Files.SideEffects
 class OIDplusPagePublicAttachments extends OIDplusPagePluginPublic {
+        /**
+         *
+         */
         const DIR_UNLOCK_FILE = 'oidplus_upload.dir';
         /**
-         * @param $dir
+         * @param string $dir
          * @return void
          * @throws OIDplusException
          */
-        private static function checkUploadDir($dir) {
+        private static function checkUploadDir(string $dir) {
                 if (!is_dir($dir)) {
                         throw new OIDplusException(_L('The attachment directory "%1" is not existing.', $dir));
+                }
                 $realdir = realpath($dir);
                         // the owner. We don't want to over-complicate it to the Windows admin.
+                }
+        }
         /**
-         * @param $dir
+         * @param string $dir
          * @return bool
          */
-        private static function isCriticalWindowsDirectory($dir): bool {
+        private static function isCriticalWindowsDirectory(string $dir): bool {
                 $dir .= '\\';
                 $windir = isset($_SERVER['SystemRoot']) ? $_SERVER['SystemRoot'].'\\' : 'C:\\Windows\\';
                 if (stripos($dir,$windir) === 0) return true;
                 return false;
+        }
         /**
-         * @param $dir
+         * @param string $dir
          * @return bool
          */
-        private static function isCriticalLinuxDirectory($dir): bool {
+        private static function isCriticalLinuxDirectory(string $dir): bool {
                 if ($dir == '/') return true;
                 $dir .= '/';
                 if (strpos($dir,'/bin/') === 0) return true;
                 if (strpos($dir,'/boot/') === 0) return true;
                 if (strpos($dir,'/dev/') === 0) return true;
                 if (strpos($dir,'/var/tmp/') === 0) return true;
                 return false;
+        }
         /**
-         * @param $id
+         * @param string|null $id
          * @return string
          * @throws OIDplusException
          */
-        public static function getUploadDir($id=null): string {
+        public static function getUploadDir(string $id=null): string {
                 // Get base path
                 $cfg = OIDplus::config()->getValue('attachment_upload_dir', '');
                 $cfg = trim($cfg);
                 if ($cfg === '') {
                         $basepath = OIDplus::localpath() . 'userdata' . DIRECTORY_SEPARATOR . 'attachments';
                 if ($doshow) $text .= $output;
+        }
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
          * @return void
          */
-        public function beforeObjectDelete($id) {}
+        public function beforeObjectDelete(string $id) {}
         /**
          * Delete the attachment folder including all files in it (note: Subfolders are not possible)
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
          * @return void
          * @throws OIDplusException
          */
-        public function afterObjectDelete($id) {
+        public function afterObjectDelete(string $id) {
                 $uploaddir = self::getUploadDir($id);
                 if ($uploaddir != '') {
                         $ary = @glob($uploaddir . DIRECTORY_SEPARATOR . '*');
                         if ($ary) foreach ($ary as $a) @unlink($a);
                         @rmdir($uploaddir);
+                }
+        }
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
-         * @param $params
+         * @param array $params
          * @return void
          */
-        public function beforeObjectUpdateSuperior($id, &$params) {}
+        public function beforeObjectUpdateSuperior(string $id, array &$params) {}
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
-         * @param $params
+         * @param array $params
          * @return void
          */
-        public function afterObjectUpdateSuperior($id, &$params) {}
+        public function afterObjectUpdateSuperior(string $id, array &$params) {}
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
-         * @param $params
+         * @param array $params
          * @return void
          */
-        public function beforeObjectUpdateSelf($id, &$params) {}
+        public function beforeObjectUpdateSelf(string $id, array &$params) {}
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
-         * @param $params
+         * @param array $params
          * @return void
          */
-        public function afterObjectUpdateSelf($id, &$params) {}
+        public function afterObjectUpdateSelf(string $id, array &$params) {}
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
-         * @param $params
+         * @param array $params
          * @return void
          */
-        public function beforeObjectInsert($id, &$params) {}
+        public function beforeObjectInsert(string $id, array &$params) {}
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.3
-         * @param $id
+         * @param string $id
-         * @param $params
+         * @param array $params
          * @return void
          */
-        public function afterObjectInsert($id, &$params) {}
+        public function afterObjectInsert(string $id, array &$params) {}
         /**
          * @param string $request
          * @return array|false
          */
                 return false;
+        }
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.4
-         * @param $id
+         * @param string $id
-         * @param $out
+         * @param array $out
          * @return void
          * @throws OIDplusException
          */
-        public function whoisObjectAttributes($id, &$out) {
+        public function whoisObjectAttributes(string $id, array &$out) {
                 $xmlns = 'oidplus-attachment-plugin';
                 $xmlschema = 'urn:oid:1.3.6.1.4.1.37476.2.5.2.4.1.95.1';
                 $xmlschemauri = OIDplus::webpath(__DIR__.'/attachments.xsd',OIDplus::PATH_ABSOLUTE);
                 $files = @glob(self::getUploadDir($id) . DIRECTORY_SEPARATOR . '*');
+        }
         /**
          * Implements interface 1.3.6.1.4.1.37476.2.5.2.3.4
-         * @param $email
+         * @param string $email
-         * @param $out
+         * @param array $out
          * @return void
          */
-        public function whoisRaAttributes($email, &$out) {}
+        public function whoisRaAttributes(string $email, array &$out) {}
+}

Subversion Repositories oidplus

oidplus/trunk/plugins/viathinksoft/publicPages/095_attachments/OIDplusPagePublicAttachments.class.php – Rev 1116 → 1130