removed .iml; modern tabs on group settings pages; beautifying code

src/messenger/messenger.iml

@@ -1,10 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<module type="JAVA_MODULE" version="4">
-  <component name="NewModuleRootManager" inherit-compiler-output="true">
-    <exclude-output />
-    <content url="file://$MODULE_DIR$" />
-    <orderEntry type="inheritedJdk" />
-    <orderEntry type="sourceFolder" forTests="false" />
-  </component>
-</module>
-

src/messenger/webim/js/source/common.js

@@ -15,7 +15,7 @@
 
 //- localized
 
-//- onComplete
+//- onComplete, showonline
 //- threadParams, servl, frequency, user, threadid, token, cssfile
 //- updaterOptions, url, company, agentservl, noclients, wroot, havemenu, showpopup, ignorectrl, istatus
 

src/messenger/webim/libs/chat.php

@@ -396,7 +396,7 @@ function load_canned_messages($locale, $groupid)
 	global $mysqlprefix;
 	$link = connect();
 	$result = select_multi_assoc(
-		"select vcvalue from ${mysqlprefix}chatresponses where locale = '" . $locale . "' " .
+		"select vcvalue from ${mysqlprefix}chatresponses where locale = '$locale' " .
 		"AND (groupid is NULL OR groupid = 0) order by vcvalue", $link);
 	if (count($result) == 0) {
 		foreach (explode("\n", getstring_('chat.predefined_answers', $locale)) as $answer) {
@@ -405,7 +405,7 @@ function load_canned_messages($locale, $groupid)
 	}
 	if ($groupid) {
 		$result2 = select_multi_assoc(
-			"select vcvalue from ${mysqlprefix}chatresponses where locale = '" . $locale . "' " .
+			"select vcvalue from ${mysqlprefix}chatresponses where locale = '$locale' " .
 			"AND groupid = $groupid order by vcvalue", $link);
 		foreach ($result as $r) {
 			$result2[] = $r;
@@ -469,7 +469,7 @@ function update_thread_access($threadid, $params, $link)
 	}
 	perform_query(
 		"update ${mysqlprefix}chatthread set $clause " .
-		"where threadid = " . $threadid, $link);
+		"where threadid = $threadid", $link);
 }
 
 function ping_thread($thread, $isuser, $istyping)
@@ -517,7 +517,7 @@ function commit_thread($threadid, $params, $link)
 	foreach ($params as $k => $v) {
 		$query .= ", " . $k . "=" . $v;
 	}
-	$query .= " where threadid = " . $threadid;
+	$query .= " where threadid = $threadid";
 
 	perform_query($query, $link);
 }

src/messenger/webim/libs/groups.php

@@ -42,9 +42,10 @@ function setup_group_settings_tabs($gid, $active)
 {
 	global $page, $webimroot, $settings;
 	if ($gid) {
+		$page['tabselected'] = $active;
 		$page['tabs'] = array(
-			getlocal("page_group.tab.main") => $active != 0 ? "$webimroot/operator/group.php?gid=$gid" : "",
-			getlocal("page_group.tab.members") => $active != 1 ? "$webimroot/operator/groupmembers.php?gid=$gid" : "",
+			array('title' => getlocal("page_group.tab.main"), 'link' => "$webimroot/operator/group.php?gid=$gid"),
+			array('title' => getlocal("page_group.tab.members"), 'link' => "$webimroot/operator/groupmembers.php?gid=$gid"),
 		);
 	} else {
 		$page['tabs'] = array();

src/messenger/webim/view/agent.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page_agent.title");
 $page['menuid'] = $page['opid'] == $page['currentopid'] ? "profile" : "operators";
 

src/messenger/webim/view/avatar.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page_avatar.title");
 $page['menuid'] = $page['opid'] == $page['currentopid'] ? "profile" : "operators";
 

src/messenger/webim/view/features.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("settings.title");
 $page['menuid'] = "settings";
 

src/messenger/webim/view/getcode_image.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page.gen_button.title");
 $page['menuid'] = "getcode";
 

src/messenger/webim/view/getcode_text.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page.gen_button.title");
 $page['menuid'] = "getcode";
 

src/messenger/webim/view/group.php

@@ -20,6 +20,8 @@
  */
 
 require_once("inc_menu.php");
+require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page.group.title");
 $page['menuid'] = "groups";
 
@@ -44,15 +46,7 @@ require_once('inc_errors.php');
 <form name="groupForm" method="post" action="<?php echo $webimroot ?>/operator/group.php">
 <input type="hidden" name="gid" value="<?php echo $page['grid'] ?>"/>
 	<div>
-<?php if($page['tabs']) { ?>
-	<ul class="tabs">
-
-<?php foreach($page['tabs'] as $k => $v) { if($v) { ?>
-	<li><a href="<?php echo $v ?>"><?php echo $k ?></a></li>
-<?php } else { ?>
-	<li class="active"><a href="#"><?php echo $k ?></a></li><?php }} ?>
-	</ul>
-<?php } ?>
+<?php print_tabbar(); ?>
 	<div class="mform"><div class="formtop"><div class="formtopi"></div></div><div class="forminner">
 
 	<div class="fieldForm">

src/messenger/webim/view/groupmembers.php

@@ -20,6 +20,8 @@
  */
 
 require_once("inc_menu.php");
+require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page.groupmembers.title");
 $page['menuid'] = "groups";
 
@@ -39,15 +41,7 @@ require_once('inc_errors.php');
 <form name="membersForm" method="post" action="<?php echo $webimroot ?>/operator/groupmembers.php">
 <input type="hidden" name="gid" value="<?php echo $page['groupid'] ?>"/>
 	<div>
-<?php if($page['tabs']) { ?>
-	<ul class="tabs">
-
-<?php foreach($page['tabs'] as $k => $v) { if($v) { ?>
-	<li><a href="<?php echo $v ?>"><?php echo $k ?></a></li>
-<?php } else { ?>
-	<li class="active"><a href="#"><?php echo $k ?></a></li><?php }} ?>
-	</ul>
-<?php } ?>
+<?php print_tabbar(); ?>
 	<div class="mform"><div class="formtop"><div class="formtopi"></div></div><div class="forminner">
 
 	<p>

src/messenger/webim/view/operator_groups.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("operator.groups.title");
 $page['menuid'] = $page['opid'] == $page['currentopid'] ? "profile" : "operators";
 

src/messenger/webim/view/performance.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("settings.title");
 $page['menuid'] = "settings";
 

src/messenger/webim/view/permissions.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("permissions.title");
 $page['menuid'] = $page['opid'] == $page['currentopid'] ? "profile" : "operators";
 

src/messenger/webim/view/settings.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("settings.title");
 $page['menuid'] = "settings";
 

src/messenger/webim/view/themes.php

@@ -21,6 +21,7 @@
 
 require_once("inc_menu.php");
 require_once("inc_tabbar.php");
+
 $page['title'] = getlocal("page.preview.title");
 $page['menuid'] = "settings";