Admin-UI:Form Handler - renderTableRow() now calls a new method: renderTableCells() to generate the table cells.

e107_handlers/admin_ui.php

@@ -5812,7 +5812,16 @@ class e_admin_ui extends e_admin_controller_ui
 			echo ADLAN_86; //Forbidden
 
 			$result = var_export($this->fields[$_name], true);
-			$this->logajax("Forbidden\nAction:".$this->getAction()."\nField:\n".$result);
+
+			$problem = array();
+			$problem['noedit'] = !empty($this->fields[$_name]['noedit']) ? 'yes' : 'no';
+			$problem['nolist'] = !empty($this->fields[$_name]['nolist']) ? 'yes' : 'no';
+			$problem['inline'] = empty($this->fields[$_name]['inline']) ? 'yes' : 'no';
+			$problem['token'] = empty($_token) ? 'yes' : 'no';
+			$problem['password'] = !password_verify(session_id(),$_token) ? 'yes' : 'no';
+
+			$result .= "\nForbidden Caused by: ".print_r($problem,true);
+			$this->logajax("Forbidden\nAction:".$this->getAction()."\nField (".$_name."):\n".$result);
 			return;
 		}
 

e107_handlers/form_handler.php

@@ -4303,19 +4303,11 @@ var_dump($select_options);*/
 	 * @param string $pid - eg. table_id
 	 * @return string
 	 */
-	function renderTableRow($fieldarray, $currentlist, $fieldvalues, $pid)
+	function renderTableCells($fieldarray, $currentlist, $fieldvalues, $pid)
 	{
+
 		$cnt = 0;
-		$ret = '';
+		$text = '';
-
-		/*$fieldarray 	= $obj->fields;
-		$currentlist 	= $obj->fieldpref;
-		$pid 			= $obj->pid;*/
-
-	//	$trclass = vartrue($fieldvalues['__trclass']) ?  ' class="'.$trclass.'"' : '';
-		unset($fieldvalues['__trclass']);
-
-		$trclass = '';
 
 		foreach ($fieldarray as $field => $data)
 		{
@@ -4337,7 +4329,7 @@ var_dump($select_options);*/
 			}
 			elseif(vartrue($data['type']) != 'method' && !$data['forced'] && !isset($fieldvalues[$field]) && $fieldvalues[$field] !== NULL)
 			{
-				$ret .= "
+				$text .= "
 					<td>
 						Not Found! ($field)
 					</td>
@@ -4353,7 +4345,6 @@ var_dump($select_options);*/
 			if($field == 'options') $tdclass = $tdclass ? $tdclass.' options' : 'options';
 
 
-            
 			// there is no other way for now - prepare user data
 			if('user' == vartrue($data['type']) /* && isset($data['readParms']['idField'])*/)
 			{
@@ -4376,17 +4367,17 @@ var_dump($select_options);*/
 					$data['readParms']['__nameval'] = $fieldvalues['user_name'];
 				}
 
-
 			}
-			$value = $this->renderValue($field, varset($fieldvalues[$field]), $data, varset($fieldvalues[$pid]));
 
+			$value = $this->renderValue($field, varset($fieldvalues[$field]), $data, varset($fieldvalues[$pid]));
 
 
 			if($tdclass)
 			{
 				$tdclass = ' class="'.$tdclass.'"';
 			}
-			$ret .= '
+
+			$text .= '
 				<td'.$tdclass.'>
 					'.$value.'
 				</td>
@@ -4397,16 +4388,43 @@ var_dump($select_options);*/
 
 		if($cnt)
 		{
-			return '
+			return $text;
-				<tr'.$trclass.' id="row-'.$fieldvalues[$pid].'">
+		}
+
+		return null;
+
+	}
+
+	/**
+	 * Render Table row and cells from field listing.
+	 *
+	 * @param array  $fieldArray  - eg. $this->fields
+	 * @param array  $fieldPref   - eg $this->fieldpref
+	 * @param array  $fieldValues - eg. $row
+	 * @param string $pid         - eg. table_id
+	 * @return string
+	 */
+	function renderTableRow($fieldArray, $fieldPref, $fieldValues, $pid)
+	{
+
+		if(!$ret = $this->renderTableCells($fieldArray, $fieldPref, $fieldValues, $pid))
+		{
+			return '';
+		}
+
+		$trclass = '';
+		//	$trclass = vartrue($fieldvalues['__trclass']) ?  ' class="'.$trclass.'"' : '';
+		unset($fieldValues['__trclass']);
+
+		return '
+				<tr'.$trclass.' id="row-' . $fieldValues[$pid].'">
 					'.$ret.'
 				</tr>
 			';
-		}
-
-		return '';
 	}
 
+
+
 	/**
 	 * Inline Token
 	 * @return string