diff --git a/gosa-core/include/class_sortableListing.inc b/gosa-core/include/class_sortableListing.inc
index 88a93bab81ef14e0be177a04dc0b0af345b21ff1..79bc25e7ab45bb4821db12350d818bef09f4e473 100644 (file)
// Save data to display
$this->setListData($data, $displayData);
// Save data to display
$this->setListData($data, $displayData);
+ // Get list of used IDs
+ if(!session::is_set('sortableListing_USED_IDS')){
+ session::set('sortableListing_USED_IDS',array());
+ }
+ $usedIds = session::get('sortableListing_USED_IDS');
+
// Generate instance wide unique ID
// Generate instance wide unique ID
- $tmp= gettimeofday();
- $this->id= 'l'.md5(microtime().$tmp['sec']);
+ $id = "";
+ while($id == "" || in_array($id, $usedIds)){
+
+ // Wait 1 msec to ensure that we definately get a new id
+ if($id != "") usleep(1);
+ $tmp= gettimeofday();
+ $id = 'l'.md5(microtime().$tmp['sec']);
+ }
+
+ // Only keep the last 10 list IDsi
+ $usedIds = array_slice($usedIds, count($usedIds) -10, 10);
+ $usedIds[] = $id;
+ session::set('sortableListing_USED_IDS',$usedIds);
+ $this->id = $id;
// Set reorderable flag
$this->reorderable= $reorderable;
// Set reorderable flag
$this->reorderable= $reorderable;
}
}
}
}
+ public function setReorderable($bool)
+ {
+ $this->reorderable= $bool;
+ }
+
+ public function setDefaultSortColumn($id)
+ {
+ $this->sortColumn = $id;
+ }
public function setListData($data, $displayData= null)
{
public function setListData($data, $displayData= null)
{
private function setDisplayData($data)
{
if (!is_array($data)) {
private function setDisplayData($data)
{
if (!is_array($data)) {
- die ("sortableList needs an array as data!");
+ trigger_error ("sortableList needs an array as data!");
}
// Transfer information
}
// Transfer information
// Preset sort orders to 'down'
for ($column= 0; $column<$this->columns; $column++) {
// Preset sort orders to 'down'
for ($column= 0; $column<$this->columns; $column++) {
- $this->sortDirection[]= true;
+ if(!isset($this->sortDirection[$column])){
+ $this->sortDirection[$column]= true;
+ }
}
}
}
}
}
// Do we need colspecs?
}
// Do we need colspecs?
- $action_width= ($this->editable?20:0) + ($this->deleteable?20:0);
+ $action_width= ($this->editable?30:0) + ($this->deleteable?30:0);
if ($this->colspecs) {
$result.= " <colgroup>\n";
for ($i= 0; $i<$this->columns; $i++) {
if ($this->colspecs) {
$result.= " <colgroup>\n";
for ($i= 0; $i<$this->columns; $i++) {
$link= "href='?plug=".$_GET['plug']."&PID=".$this->id."&act=SORT_$i'";
$sorter= "";
if ($i == $this->sortColumn){
$link= "href='?plug=".$_GET['plug']."&PID=".$this->id."&act=SORT_$i'";
$sorter= "";
if ($i == $this->sortColumn){
- $sorter= " ".image("images/lists/sort-".($this->sortDirection[$i]?"up":"down").".png", null, $this->sortDirection[$i]?_("Up"):_("Down"));
+ $sorter= " ".image("images/lists/sort-".($this->sortDirection[$i]?"up":"down").".png", null, $this->sortDirection[$i]?_("Sort ascending"):_("Sort descending"));
}
if ($this->reorderable) {
}
if ($this->reorderable) {
$first= " style='border:0'";
foreach ($this->displayData[$row] as $column) {
$first= " style='border:0'";
foreach ($this->displayData[$row] as $column) {
- $result.= " <td$editable$first>".$column."</td>\n";
- $first= "";
+
+ // Do NOT use the onClick statement for columns that contain links or buttons.
+ if(preg_match("<.*type=.submit..*>", $column) || preg_match("<a.*href=.*>", $column)){
+ $result.= " <td$first>".$column."</td>\n";
+ }else{
+ $result.= " <td$editable$first>".$column."</td>\n";
+ }
+ $first= "";
}
if ($action_width) {
}
if ($action_width) {
public function update()
{
public function update()
{
- // Do not do anything if this is not our PID, or there's even no PID available...
- if(!isset($_REQUEST['PID']) || $_REQUEST['PID'] != $this->id) {
-# return;
- }
// Filter GET with "act" attributes
// Filter GET with "act" attributes
- if (!$this->reorderable && isset($_GET['act'])) {
- $key= validate($_GET['act']);
- if (preg_match('/^SORT_([0-9]+)$/', $key, $match)) {
- // Switch to new column or invert search order?
- $column= $match[1];
- if ($this->sortColumn != $column) {
- $this->sortColumn= $column;
- } else {
- $this->sortDirection[$column]= !$this->sortDirection[$column];
- }
+ if (!$this->reorderable){
+ if(isset($_GET['act']) && isset($_GET['PID']) && $this->id == $_GET['PID']) {
+
+ $key= validate($_GET['act']);
+ if (preg_match('/^SORT_([0-9]+)$/', $key, $match)) {
+
+ // Switch to new column or invert search order?
+ $column= $match[1];
+ if ($this->sortColumn != $column) {
+ $this->sortColumn= $column;
+ } else {
+ $this->sortDirection[$column]= !$this->sortDirection[$column];
+ }
- // Update mapping according to sort parameters
- $this->sortData();
+ }
}
}
+
+ // Update mapping according to sort parameters
+ $this->sortData();
}
}
}
}
public function save_object()
{
// Do not do anything if this is not our PID, or there's even no PID available...
public function save_object()
{
// Do not do anything if this is not our PID, or there's even no PID available...
- if(!isset($_REQUEST['PID']) || $_REQUEST['PID'] != $this->id) {
-# return;
+ if(isset($_REQUEST['PID']) && $_REQUEST['PID'] != $this->id) {
+ return;
}
// Do not do anything if we're not posted - or have no permission
}
// Do not do anything if we're not posted - or have no permission
public function sortData()
{
public function sortData()
{
+ if(!count($this->data)) return;
+
// Extract data
$tmp= array();
foreach($this->displayData as $item) {
// Extract data
$tmp= array();
foreach($this->displayData as $item) {
$this->mapping[]= count($this->mapping);
$this->modified= true;
$this->mapping[]= count($this->mapping);
$this->modified= true;
+ // Find the number of coluns
+ reset($this->displayData);
+ $first= current($this->displayData);
+ if (is_array($first)) {
+ $this->columns= count($first);
+ } else {
+ $this->columns= 1;
+ }
+
+ // Preset sort orders to 'down'
+ for ($column= 0; $column<$this->columns; $column++) {
+ if(!isset($this->sortDirection[$column])){
+ $this->sortDirection[$column]= true;
+ }
+ }
+
+
// Sort data after we've added stuff
$this->sortData();
}
// Sort data after we've added stuff
$this->sortData();
}