From d9e718508cbafb1b55a5d631217e213aef2f22a7 Mon Sep 17 00:00:00 2001
From: jmontoyaa <gugli100@gmail.com>
Date: Wed, 27 Apr 2016 13:22:04 +0200
Subject: [PATCH] WIP vchamilo plugin

Fix UI
---
 plugin/vchamilo/ajax/service.php     |  4 ++--
 plugin/vchamilo/js/ajax.js           |  6 ++----
 plugin/vchamilo/views/syncparams.php | 30 ++++++++++++++++++----------
 3 files changed, 24 insertions(+), 16 deletions(-)

diff --git a/plugin/vchamilo/ajax/service.php b/plugin/vchamilo/ajax/service.php
index 64e766e214..234c550a6e 100644
--- a/plugin/vchamilo/ajax/service.php
+++ b/plugin/vchamilo/ajax/service.php
@@ -14,8 +14,8 @@ api_protect_admin_script();
 if ($action == 'syncthis') {
     $res = include_once api_get_path(SYS_PLUGIN_PATH).'vchamilo/views/syncparams.controller.php';
     if (!$res) {
-        echo '<span class="ok">Success</span>';
+        echo '<span class="label label-success">Success</span>';
     } else {
-        echo '<span class="failed">Failure<br/>'.$errors.'</span>';
+        echo '<span class="label label-danger">Failure<br/>'.$errors.'</span>';
     }
 }
\ No newline at end of file
diff --git a/plugin/vchamilo/js/ajax.js b/plugin/vchamilo/js/ajax.js
index f4460596d5..eb1a3340f1 100644
--- a/plugin/vchamilo/js/ajax.js
+++ b/plugin/vchamilo/js/ajax.js
@@ -1,13 +1,11 @@
 
 function ajax_sync_setting(wwwroot, settingid) {
-
     spare = $('#row_'+settingid).html();
     formobj = document.forms['settingsform'];
-
     url = wwwroot+'plugin/vchamilo/ajax/service.php?what=syncthis&settingid='+settingid+'&value='+encodeURIComponent(formobj.elements['value_'+settingid].value);
-    if (formobj.elements['del_'+settingid].checked) {
+    /*if (formobj.elements['del_'+settingid].checked) {
         url += '&del=1';
-    }
+    }*/
 
     $('#row_'+settingid).html('<td colspan="7"><img src="'+wwwroot+'plugin/vchamilo/pix/ajax_waiter.gif" /></td>');
 
diff --git a/plugin/vchamilo/views/syncparams.php b/plugin/vchamilo/views/syncparams.php
index 537d7f9347..315369883f 100644
--- a/plugin/vchamilo/views/syncparams.php
+++ b/plugin/vchamilo/views/syncparams.php
@@ -25,27 +25,35 @@ $column = 0;
 $row = 0;
 
 // $table->set_additional_parameters($parameters);
-$headers = array('', $plugininstance->get_lang('variable'), $plugininstance->get_lang('subkey'), $plugininstance->get_lang('category'), $plugininstance->get_lang('accessurl'), $plugininstance->get_lang('value'), '');
+$headers = array(
+    '',
+    $plugininstance->get_lang('variable').' ['.$plugininstance->get_lang('subkey').']',
+    $plugininstance->get_lang('category'),
+    $plugininstance->get_lang('accessurl'),
+    $plugininstance->get_lang('value'),
+    '',
+);
 $attrs = array('center' => 'left');
 $table->addRow($headers, $attrs, 'th');
 
 foreach ($settings as $param) {
+
+    if ($param['subkey'] == 'vchamilo') {
+        continue;
+    }
     // $check = '<input type="checkbox" name="sync_'.$param->id.'" value="'.$param->selected_value.'" />';
+    //<input type="checkbox" name="del_'.$param['id'].'" value="1" title="'.$plugininstance->get_lang('deleteifempty').'" />
     $check = '';
     $attrs = array('center' => 'left');
     $syncthisbutton = '<input type="button" name="syncthis" value="'.$plugininstance->get_lang('syncthis').'" onclick="ajax_sync_setting(\''.$_configuration['root_web'].'\', \''.$param['id'].'\')" /> 
-        <input type="checkbox" name="del_'.$param['id'].'" value="1" title="'.$plugininstance->get_lang('deleteifempty').'" /> <span id="res_'.$param['id'].'"></span>';
+         <span id="res_'.$param['id'].'"></span>';
     $data = array(
         $check,
-        $param['variable'],
-        $param['subkey'],
+        isset($param['subkey']) && !empty($param['subkey']) ? $param['variable'].' ['.$param['subkey'].']' : $param['variable'],
         $param['category'],
         $param['access_url'],
-        '<input type="text" name="value_'.$param['id'].'" value="'.htmlspecialchars(
-            $param['selected_value'],
-            ENT_COMPAT,
-            'UTF-8'
-        ).'" />',
+        '<input type="text" name="value_'.$param['id'].'" value="'.htmlspecialchars($param['selected_value'], ENT_COMPAT, 'UTF-8' ).'" />'.
+        '<br />Master value: '.$param['selected_value'],
         $syncthisbutton,
     );
     $row = $table->addRow($data, $attrs, 'td');
@@ -59,11 +67,13 @@ $content .=  $table->toHtml();
 $content .=  '</form>';
 
 $actions = '';
-$message = '';
+
+Display::addFlash(Display::return_message($plugininstance->get_lang('Sync your master settings to all instances.')));
 
 $message = require_js('ajax.js', 'vchamilo', true);
 
 $interbreadcrumb[] = array('url' => 'manage.php', 'name' => get_lang('VChamilo'));
+
 $tpl = new Template($plugininstance->get_lang('sync_settings'), true, true, false, true, false);
 $tpl->assign('actions', $actions);
 $tpl->assign('message', $message);