diff --git a/ajax/settings.php b/ajax/settings.php index ee085a6..eabdfd3 100644 --- a/ajax/settings.php +++ b/ajax/settings.php @@ -75,20 +75,40 @@ if(isset($_POST['appname']) && ($_POST['appname'] === 'user_sql') && isset($_POS 'data' => array('message' => $l -> t('The selected SQL table '.$_POST['sql_table'].' does not exist!')))); break; } + if(!empty($_POST['sql_group_table']) && !$helper->verifyTable($parameters, $_POST['sql_driver'], $_POST['sql_group_table'])) + { + $response->setData(array('status' => 'error', + 'data' => array('message' => $l -> t('The selected SQL table '.$_POST['sql_group_table'].' does not exist!')))); + break; + } // Retrieve all column settings $columns = array(); + $group_columns = array(); foreach($params as $param) { if(strpos($param, 'col_') === 0) { if(isset($_POST[$param]) && $_POST[$param] !== '') - $columns[] = $_POST[$param]; + { + if(strpos($param, 'col_group_') === 0) + { + $group_columns[] = $_POST[$param]; + } + else + { + $columns[] = $_POST[$param]; + } + } } } // Check if the columns exist $status = $helper->verifyColumns($parameters, $_POST['sql_driver'], $_POST['sql_table'], $columns); + if(!empty($_POST['sql_group_table']) && $status === true) + { + $status = $helper->verifyColumns($parameters, $_POST['sql_driver'], $_POST['sql_group_table'], $group_columns); + } if($status !== true) { $response->setData(array('status' => 'error', diff --git a/appinfo/app.php b/appinfo/app.php index 2617fb9..28ac922 100644 --- a/appinfo/app.php +++ b/appinfo/app.php @@ -22,10 +22,13 @@ */ require_once __DIR__ . '/../user_sql.php'; +require_once __DIR__ . '/../group_sql.php'; \OCP\App::registerAdmin('user_sql','settings'); $backend = new \OCA\user_sql\OC_USER_SQL; +$group_backend = new \OCA\user_sql\OC_GROUP_SQL; // register user backend \OC_User::useBackend($backend); +\OC::$server->getGroupManager()->addBackend($group_backend); diff --git a/appinfo/info.xml b/appinfo/info.xml index 2b557d1..d2b6911 100644 --- a/appinfo/info.xml +++ b/appinfo/info.xml @@ -15,5 +15,6 @@ auth + diff --git a/js/settings.js b/js/settings.js index a77bffb..034cc2e 100644 --- a/js/settings.js +++ b/js/settings.js @@ -17,7 +17,7 @@ user_sql.adminSettingsUI = function() $('#sqlDiv').tabs(); // Attach auto-completion to all column fields - $('#col_username, #col_password, #col_displayname, #col_active, #col_email, #col_gethome').autocomplete({ + $('#col_username, #col_password, #col_displayname, #col_active, #col_email, #col_gethome, #col_group_name, #col_group_username').autocomplete({ source: function(request, response) { var post = $('#sqlForm').serializeArray(); @@ -56,7 +56,7 @@ user_sql.adminSettingsUI = function() }); // Attach auto-completion to all table fields - $('#sql_table').autocomplete({ + $('#sql_table, #sql_group_table').autocomplete({ source: function(request, response) { var post = $('#sqlForm').serializeArray(); diff --git a/lib/helper.php b/lib/helper.php index 7a3023f..18177a3 100644 --- a/lib/helper.php +++ b/lib/helper.php @@ -64,7 +64,10 @@ class Helper { 'set_mail_sync_mode', 'set_enable_gethome', 'set_gethome_mode', - 'set_gethome' + 'set_gethome', + 'sql_group_table', + 'col_group_username', + 'col_group_name' ); return $params; @@ -173,6 +176,18 @@ class Helper { case 'mysqlPassword': $query = "SELECT PASSWORD(:pw);"; break; + + case 'getUserGroups': + $query = "SELECT ".$this->settings['col_group_name']." FROM ".$this->settings['sql_group_table']." WHERE ".$this->settings['col_group_username']." = :uid"; + break; + + case 'getGroups': + $query = "SELECT distinct ".$this->settings['col_group_name']." FROM ".$this->settings['sql_group_table']; + break; + + case 'getGroupUsers': + $query = "SELECT distinct ".$this->settings['col_group_username']." FROM ".$this->settings['sql_group_table']." WHERE ".$this->settings['col_group_name']." = :gid"; + break; } if(isset($limits['limit']) && $limits['limit'] !== null) @@ -265,7 +280,7 @@ class Helper { if(!in_array($col, $columns, true)) { $res = false; - $err .= $col.' '; + $err .= $table.'.'.$col.' '; } } if($res) @@ -299,13 +314,19 @@ class Helper { try { $conn = $cm -> getConnection($sql_driver, $parameters); $platform = $conn -> getDatabasePlatform(); - $query = $platform -> getListTablesSQL(); - $result = $conn -> executeQuery($query); + + $queries = array( + 'Tables_in_'.$parameters['dbname'] => $platform -> getListTablesSQL(), + 'TABLE_NAME' => $platform -> getListViewsSQL($parameters['dbname'])); $ret = array(); - while($row = $result -> fetch()) + foreach($queries as $field => $query) { - $name = $row['Tables_in_'.$parameters['dbname']]; - $ret[] = $name; + $result = $conn -> executeQuery($query); + while($row = $result -> fetch()) + { + $name = $row[$field]; + $ret[] = $name; + } } return $ret; } diff --git a/templates/settings.php b/templates/settings.php index c555c5c..11dd10b 100644 --- a/templates/settings.php +++ b/templates/settings.php @@ -20,6 +20,7 @@ $cfgClass = $ocVersion >= 7 ? 'section' : 'personalblock';
  • t('E-Mail Settings')); ?>
  • t('Domain Settings')); ?>
  • t('getHome Settings')); ?>
  • +
  • t('Groups Settings')); ?>
  • @@ -155,6 +156,15 @@ $cfgClass = $ocVersion >= 7 ? 'section' : 'personalblock'; t('You can use the placeholders %%u to specify the user ID (before appending the default domain), %%ud to specify the user ID (after appending the default domain) and %%d to specify the default domain')); ?>

    +
    +

    + +

    + +

    + +
    +