Passed
Push — scrutinizer-code-quality ( 09f5a1...c4c5fb )
by Adam
56:05 queued 14:08
created
modules/ModuleBuilder/parsers/relationships/ManyToOneRelationship.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -127,6 +127,9 @@
 block discarded – undo
127 127
         return $this->one_to_many->buildRelationshipMetaData();
128 128
     }
129 129
     
130
+    /**
131
+     * @param string $relationshipName
132
+     */
130 133
     public function setName ($relationshipName)
131 134
     {
132 135
         parent::setName($relationshipName);
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/relationships/UndeployedRelationships.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -310,11 +310,17 @@
 block discarded – undo
310 310
         }
311 311
     }
312 312
 
313
+    /**
314
+     * @param AbstractRelationship $relationship
315
+     */
313 316
     private function addFieldsToUndeployedLayouts ($relationship)
314 317
     {
315 318
         return $this->updateUndeployedLayout ( $relationship, true ) ;
316 319
     }
317 320
 
321
+    /**
322
+     * @param AbstractRelationship $relationship
323
+     */
318 324
     private function removeFieldsFromUndeployedLayouts ($relationship)
319 325
     {
320 326
         return $this->updateUndeployedLayout ( $relationship, false ) ;
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/StandardField.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@
 block discarded – undo
80 80
      * Adds a custom field using a field object
81 81
      *
82 82
      * @param Field Object $field
83
-     * @return boolean
83
+     * @return false|null
84 84
      */
85 85
     function addFieldObject(&$field){
86 86
         global $dictionary, $beanList;
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/views/AbstractMetaDataImplementation.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -159,6 +159,10 @@
 block discarded – undo
159 159
 	}
160 160
 
161 161
 	
162
+	/**
163
+	 * @param string $filename
164
+	 * @param string $view
165
+	 */
162 166
 	protected function _loadFromPopupFile ($filename, $mod, $view, $forSave = false)
163 167
 	{
164 168
 		// BEGIN ASSERTIONS
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/views/DeployedMetaDataImplementation.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -361,6 +361,9 @@
 block discarded – undo
361 361
 		return $this->recursiveVariableReplace($defs, $module, $var_values);
362 362
 	}
363 363
 
364
+	/**
365
+	 * @return string
366
+	 */
364 367
 	public function getModuleDir(){
365 368
 		return $this->module_dir;
366 369
 	}
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/views/GridLayoutMetaDataParser.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -849,6 +849,9 @@  discard block
 block discarded – undo
849 849
         return false;
850 850
     }
851 851
     
852
+    /**
853
+     * @param boolean $useTabs
854
+     */
852 855
     public function setUseTabs($useTabs){
853 856
         $this->_viewdefs  [ 'templateMeta' ]['useTabs'] = $useTabs;
854 857
     }
@@ -865,7 +868,7 @@  discard block
 block discarded – undo
865 868
     
866 869
     /**
867 870
      * Sync DetailView & EditView. This should only be set on the EditView
868
-     * @param bool $syncViews
871
+     * @param boolean $syncDetailEditViews
869 872
      */
870 873
     public function setSyncDetailEditViews($syncDetailEditViews){
871 874
         $this->_viewdefs  [ 'templateMeta' ]['syncDetailEditViews'] = $syncDetailEditViews;
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/views/History.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -148,6 +148,10 @@
 block discarded – undo
148 148
      * Add an item to the history
149 149
      * @return String   A GMT Unix timestamp for this newly added item
150 150
      */
151
+
152
+    /**
153
+     * @param string $path
154
+     */
151 155
     public function append ($path)
152 156
     {
153 157
         // make sure we don't have a duplicate filename - highly unusual as two people should not be using Studio/MB concurrently, but when testing quite possible to do two appends within one second...
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/views/ListLayoutMetaDataParser.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -72,6 +72,10 @@
 block discarded – undo
72 72
      * @param string moduleName     The name of the module to which this listview belongs
73 73
      * @param string packageName    If not empty, the name of the package to which this listview belongs
74 74
      */
75
+
76
+    /**
77
+     * @param string $view
78
+     */
75 79
     function __construct ($view , $moduleName , $packageName = '')
76 80
     {
77 81
         $GLOBALS [ 'log' ]->debug ( get_class ( $this ) . ": __construct()" ) ;
Please login to merge, or discard this patch.
modules/ModuleBuilder/parsers/views/MetaDataParserInterface.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -42,6 +42,9 @@
 block discarded – undo
42 42
 interface MetaDataParserInterface
43 43
 {
44 44
 
45
+    /**
46
+     * @return void
47
+     */
45 48
     public function handleSave () ;
46 49
 
47 50
     public function getLayout () ;
Please login to merge, or discard this patch.