Index: branches/2.8.x/CHANGELOG
===================================================================
--- branches/2.8.x/CHANGELOG	(revision 1739)
+++ branches/2.8.x/CHANGELOG	(revision 1740)
@@ -13,6 +13,12 @@
 
 
 
+05 Sep-2012 Build 1740 Dietmar Woellbrink (Luisehahne)
+! frontend.functions.php remove return $content in page_content(), 
+  incompatible with search logic
+! /include/jquery forgot jquery version.txt 
+# /form/view.php fixed wrong compare syntax 
+! group management, design style, set max-heights for permission boxes
 05 Sep-2012 Build 1739 Dietmar Woellbrink (Luisehahne)
 ! update jquery ui to version 1.8.23
 05 Sep-2012 Build 1738 Dietmar Woellbrink (Luisehahne)
Index: branches/2.8.x/wb/include/jquery/version.txt
===================================================================
--- branches/2.8.x/wb/include/jquery/version.txt	(revision 1739)
+++ branches/2.8.x/wb/include/jquery/version.txt	(revision 1740)
@@ -1,2 +1,2 @@
 jQuery 1.8.1
-jQuery UI 1.8.17
\ No newline at end of file
+jQuery UI 1.8.23
\ No newline at end of file
Index: branches/2.8.x/wb/admin/skel/themes/htt/groups_form.htt
===================================================================
--- branches/2.8.x/wb/admin/skel/themes/htt/groups_form.htt	(revision 1739)
+++ branches/2.8.x/wb/admin/skel/themes/htt/groups_form.htt	(revision 1740)
@@ -78,7 +78,7 @@
 			<!-- BEGIN show_cmd_hidden_advanced_permission_list_block -->
 						<input name="{SYS_NAME}" type="hidden" value="{SYS_VALUE}" />
 			<!-- END show_cmd_hidden_advanced_permission_list_block -->
-						<div class="mbox curved min_h100 floatbox">
+						<div class="mbox curved min_h100 floatbox" style="height: 180px;">
 							<h6>{FILESYSTEM_PERMISSIONS} {MENU_PAGES}</h6>
 							<ul>
 								<li class="c100">
@@ -115,7 +115,7 @@
 				</div>
 				<div class="cleft col_width33">
 					<div class="col_content">
-						<div class="mbox curved min_h100 floatbox">
+						<div class="mbox curved min_h100 floatbox" style="height: 180px;">
 							<h6>{TEXT_FILESYSTEM_PERMISSIONS} {MENU_MEDIA}</h6>
 							<ul>
 								<li class="c100">
@@ -147,7 +147,7 @@
 				</div>
 				<div class="cleft col_width33">
 					<div class="col_content">
-						<div class="mbox curved min_h100 floatbox">
+						<div class="mbox curved min_h100 floatbox" style="height: 180px;">
 							<h6>{FILESYSTEM_PERMISSIONS} {MENU_SETTINGS}</h6>
 							<ul>
 								<li class="c100">
@@ -294,9 +294,10 @@
 		<!-- END show_cmd_advanced_permission_block -->
 				<hr class="clear" />
 				<div class="cleft col_width33">
+                        {HEADER_MODULE_FUNCTION}
 					<div class="col_content">
-                        {HEADER_MODULE_FUNCTION}
-						<div class="mbox curved min_h100" style="overflow: hidden;">
+						<div class="mbox curved min_h100" style="overflow: auto; margin-top: 0;height: 300px;">
+                        <div style="overflow: hidden;">
 							<ul style="margin: 0; padding: 0; ">
 		<!-- BEGIN pages_module_list_block -->
 								<li class="c100">
@@ -305,17 +306,18 @@
 									<input class="radio" type="checkbox" name="module_permissions[]" id="m_{MOD_VALUE}" value="{MOD_VALUE}" {MOD_CHECKED} />
 									<label class="" for="m_{MOD_VALUE}">{MOD_NAME}</label>
 								</div>
-
-			</li>
+                    			</li>
 		<!-- END pages_module_list_block -->
-		</ul>
+                    		</ul>
 						</div>
+						</div>
 					</div>
 				</div>
 				<div class="cleft col_width33">
-					<div class="col_content">
                         {HEADER_MODULE_FUNCTION}
-						<div class="mbox curved min_h100" style="overflow: hidden;">
+					<div class="col_content" >
+						<div class="mbox curved min_h100" style="overflow: auto; margin-top: 0;height: 300px">
+                        <div style="overflow: hidden;">
 							<ul style="margin: 0; padding: 0; ">
 		<!-- BEGIN tools_module_list_block -->
 								<li class="c100">
@@ -328,12 +330,14 @@
 		<!-- END tools_module_list_block -->
 							</ul>
 						</div>
+						</div>
 					</div>
 				</div>
 				<div class="cleft col_width33">
-					<div class="col_content">
                         {HEADER_TEMPLATE_FUNCTION}
-						<div class="mbox curved min_h100" style="overflow: hidden;">
+					<div class="col_content" >
+						<div class="mbox curved min_h100" style="overflow: auto; margin-top: 0;height: 300px">
+                        <div style="overflow: hidden;">
 							<ul style="margin: 0; padding: 0; ">
 			<!-- BEGIN template_list_block -->
 								<li class="c100">
@@ -340,10 +344,11 @@
                                     {TEMPLATE_FUNCTION}
 									<input class="radio" type="checkbox" name="template_permissions[]" id="m_{TMP_VALUE}" value="{TMP_VALUE}" {TMP_CHECKED} />
 									<label class="" for="m_{TMP_VALUE}">{TMP_NAME}</label>
-			</li>
+                    			</li>
 			<!-- END template_list_block -->
-		</ul>
+                    		</ul>
 						</div>
+						</div>
 					</div>
 				</div>
 				<hr class="clear" />
Index: branches/2.8.x/wb/admin/interface/version.php
===================================================================
--- branches/2.8.x/wb/admin/interface/version.php	(revision 1739)
+++ branches/2.8.x/wb/admin/interface/version.php	(revision 1740)
@@ -51,5 +51,5 @@
 
 // check if defined to avoid errors during installation (redirect to admin panel fails if PHP error/warnings are enabled)
 if(!defined('VERSION')) define('VERSION', '2.8.3');
-if(!defined('REVISION')) define('REVISION', '1739');
+if(!defined('REVISION')) define('REVISION', '1740');
 if(!defined('SP')) define('SP', '');
Index: branches/2.8.x/wb/framework/frontend.functions.php
===================================================================
--- branches/2.8.x/wb/framework/frontend.functions.php	(revision 1739)
+++ branches/2.8.x/wb/framework/frontend.functions.php	(revision 1740)
@@ -240,7 +240,7 @@
 	 * @param boolean true to print $content, false return $content
 	 * @return void
 	 */
-	function page_content($block = 1, $echo=true) {
+	function page_content($block = 1) {
 		// Get outside objects
 		global $TEXT,$MENU,$HEADING,$MESSAGE;
 		global $globals;
@@ -247,6 +247,7 @@
 		global $database;
 		global $wb;
 		$admin = $wb;
+        $retVal = '';
 		if ($wb->page_access_denied==true) {
 	        echo $MESSAGE['FRONTEND_SORRY_NO_VIEWING_PERMISSIONS'];
 			return;
@@ -266,23 +267,15 @@
 		if(!defined('PAGE_CONTENT') OR $block!=1)
         {
 			$page_id = intval($wb->page_id);
-/* move to class.frontend
-            // set session variable to save page_id only if PAGE_CONTENT is empty
-            $_SESSION['PAGE_ID'] = !isset($_SESSION['PAGE_ID']) ? $page_id : $_SESSION['PAGE_ID'];
-            // set to new value if page_id changed and not 0
-            if(($page_id != 0) && ($_SESSION['PAGE_ID'] <> $page_id))
-            {
-	            $_SESSION['PAGE_ID'] = $page_id;
-            }
-*/
+
 		// First get all sections for this page
 			$sql  = 'SELECT `section_id`, `module`, `publ_start`, `publ_end` ';
 			$sql .= 'FROM `'.TABLE_PREFIX.'sections` ';
 			$sql .= 'WHERE `page_id`='.$page_id.' AND `block`='.$block.' ';
 			$sql .= 'ORDER BY `position`';
-			if( !($query_sections = $database->query($sql)) ) { return; }
+			if( !($oSections = $database->query($sql)) ) { return; }
 		// If none were found, check if default content is supposed to be shown
-			if($query_sections->numRows() == 0) {
+			if($oSections->numRows() == 0) {
 				if($wb->default_block_content == 'none') { return; }
 				if (is_numeric($wb->default_block_content)) {
 					$page_id = $wb->default_block_content;
@@ -289,16 +282,18 @@
 				} else {
 					$page_id = $wb->default_page_id;
 				}
+
 				$sql  = 'SELECT `section_id`, `module`, `publ_start`, `publ_end` ';
 				$sql .= 'FROM `'.TABLE_PREFIX.'sections` ';
 				$sql .= 'WHERE `page_id`='.$page_id.' AND `block`='.$block.' ';
 				$sql .= 'ORDER BY `position`';
-				if( !($query_sections = $database->query($sql)) ) { return; }
+				if( !($oSections = $database->query($sql)) ) { return; }
 				// Still no cotent found? Give it up, there's just nothing to show!
-				if($query_sections->numRows() == 0) { return; }
+				if($oSections->numRows() == 0) { return; }
 			}
+
 			// Loop through them and include their module file
-			while($section = $query_sections->fetchRow()) {
+			while($section = $oSections->fetchRow()) {
 				// skip this section if it is out of publication-date
 				$now = time();
 				if( !(($now<=$section['publ_end'] || $section['publ_end']==0) && ($now>=$section['publ_start'] || $section['publ_start']==0)) ) {
@@ -306,35 +301,37 @@
 				}
 				$section_id = $section['section_id'];
 				$module = $section['module'];
-				// make a anchor for every section.
-				if(defined('SEC_ANCHOR') && SEC_ANCHOR!='') {
-					echo '<a class="section_anchor" id="'.SEC_ANCHOR.$section_id.'" name="'.SEC_ANCHOR.$section_id.'"></a>';
-				}
                 // check if module exists - feature: write in errorlog
 				if(file_exists(WB_PATH.'/modules/'.$module.'/view.php')) {
 				// fetch content -- this is where to place possible output-filters (before highlighting)
 					ob_start(); // fetch original content
+				// make a anchor for every section
+    				if(defined('SEC_ANCHOR') && SEC_ANCHOR!='') {
+    					echo "\n".'<a class="section_anchor" id="'.SEC_ANCHOR.$section_id.'" name="'.SEC_ANCHOR.$section_id.'"></a>'."\n";
+    				}
 					require(WB_PATH.'/modules/'.$module.'/view.php');
 					$content = ob_get_clean();
 				} else {
 					continue;
 				}
+
 				// highlights searchresults
-				if(isset($_GET['searchresult']) && is_numeric($_GET['searchresult']) && !isset($_GET['nohighlight']) && isset($_GET['sstring']) && !empty($_GET['sstring'])) {
+				if(isset($_GET['searchresult']) && is_numeric($_GET['searchresult']) && !isset($_GET['nohighlight']) && isset($_GET['sstring']) && !empty($_GET['sstring']))
+                {
 					$arr_string = explode(" ", $_GET['sstring']);
-					if($_GET['searchresult']==2) { // exact match
+					if($_GET['searchresult']==2) {  //exact match
 						$arr_string[0] = str_replace("_", " ", $arr_string[0]);
 					}
 					echo search_highlight($content, $arr_string);
-				} elseif($echo==true) {
+				} else {
 					echo $content;
-				} else {
-				    return $content;
 				}
 			}
 		} else {
 			require(PAGE_CONTENT);
 		}
+    return $retVal;
+
 	}
 }
 
Index: branches/2.8.x/wb/modules/form/view.php
===================================================================
--- branches/2.8.x/wb/modules/form/view.php	(revision 1739)
+++ branches/2.8.x/wb/modules/form/view.php	(revision 1740)
@@ -221,7 +221,7 @@
 					$options = explode(',', $value);
 					array_walk($options, 'make_option', (isset($_SESSION['field'.$field_id])?$_SESSION['field'.$field_id]:array()));
 					$field['extra'] = explode(',',$field['extra']);
-					$field['extra'][1] = ($field['extra'][1]='multiple') ? $field['extra'][1].'="'.$field['extra'][1].'"' : '';
+					$field['extra'][1] = ($field['extra'][1]=='multiple') ? $field['extra'][1].'="'.$field['extra'][1].'"' : '';
 					$values[] = '<select name="field'.$field_id.'[]" id="field'.$field_id.'" size="'.$field['extra'][0].'" '.$field['extra'][1].' class="frm-select">'.implode($options).'</select>'.PHP_EOL;
 				} elseif($field['type'] == 'heading') {
 					$vars[] = '{FIELD}';
