cf: menu attirbutes for list sorting

src/curses/ui_call_list.c

@@ -101,6 +101,7 @@ call_list_create(ui_t *ui)
 
     // Form starts inactive
     call_list_form_activate(ui, 0);
+    info->menu_active = 0;
 
     // Calculate available printable area
     info->list_win = subwin(ui->win, ui->height - 5, ui->width, 4, 0);
@@ -210,7 +211,19 @@ call_list_draw_header(ui_t *ui)
     // Draw columns titles
     wattron(ui->win, A_BOLD | COLOR_PAIR(CP_DEF_ON_CYAN));
     mvwprintw(ui->win, 3, 0, "%*s", ui->width, "");
-    for (colpos = 6, i = 0; i < info->columncnt; i++) {
+
+    // Draw separator line
+    if (info->menu_active) {
+        colpos = 18;
+        mvwprintw(ui->win, 3, 0, "Sort by");
+        wattron(ui->win, A_BOLD | COLOR_PAIR(CP_CYAN_ON_BLACK));
+        mvwprintw(ui->win, 3, 11, "%*s", 1, "");
+        wattron(ui->win, A_BOLD | COLOR_PAIR(CP_DEF_ON_CYAN));
+    } else {
+        colpos = 6;
+    }
+
+    for (i = 0; i < info->columncnt; i++) {
         // Get current column width
         collen = info->columns[i].width;
         // Get current column title
@@ -399,6 +412,7 @@ call_list_draw_list(ui_t *ui)
     ui_scrollbar_draw(info->scroll);
 
     // Refresh the list
+    if (!info->menu_active)
         wnoutrefresh(info->list_win);
 }
 
@@ -501,7 +515,6 @@ call_list_handle_key(ui_t *ui, int key)
     int action = -1;
     sip_call_t *call;
     sip_sort_t sort;
-    int i;
 
     // Sanity check, this should not happen
     if (!(info  = call_list_info(ui)))
@@ -511,6 +524,9 @@ call_list_handle_key(ui_t *ui, int key)
     if (info->form_active)
         return call_list_handle_form_key(ui, key);
 
+    if (info->menu_active)
+        return call_list_handle_menu_key(ui, key);
+
     // Get window of call list panel
     WINDOW *list_win = info->list_win;
     getmaxyx(list_win, listh, listw);
@@ -626,29 +642,9 @@ call_list_handle_key(ui_t *ui, int key)
                 sort.asc = (sort.asc) ? false : true;
                 sip_set_sort_options(sort);
                 break;
-            case ACTION_SORT_PREV:
-                // Change sort attribute
-                sort = sip_sort_options();
-                for (i = 0; i < info->columncnt; i++) {
-                    if (info->columns[i].id == sort.by &&  i != 0) {
-                        sort.by = info->columns[i-1].id;
-                        break;
-                    }
-                }
-                sip_set_sort_options(sort);
-                break;
             case ACTION_SORT_NEXT:
-                // Change sort attribute
+            case ACTION_SORT_PREV:
-                sort = sip_sort_options();
+                call_list_select_sort_attribute(ui);
-                for (i = 0; i < info->columncnt; i++) {
-                    if (info->columns[i].id == sort.by) {
-                        if (i + 1 != info->columncnt) {
-                            sort.by = info->columns[i+1].id;
-                            break;
-                        }
-                    }
-                }
-                sip_set_sort_options(sort);
                 break;
             case ACTION_PREV_SCREEN:
                 // Handle quit from this screen unless requested
@@ -777,6 +773,77 @@ call_list_handle_form_key(ui_t *ui, int key)
     return (action == ERR) ? KEY_NOT_HANDLED : KEY_HANDLED;
 }
 
+int
+call_list_handle_menu_key(ui_t *ui, int key)
+{
+    MENU *menu;
+    ITEM *current;
+    int current_idx;
+    int action = -1;
+    sip_sort_t sort;
+    enum sip_attr_id id;
+
+    // Get panel information
+    call_list_info_t *info = call_list_info(ui);
+
+    menu = info->menu;
+    current = current_item(menu);
+    current_idx = item_index(current);
+
+      // Check actions for this key
+      while ((action = key_find_action(key, action)) != ERR) {
+          // Check if we handle this action
+          switch (action) {
+              case ACTION_DOWN:
+                  menu_driver(menu, REQ_DOWN_ITEM);
+                  break;
+              case ACTION_UP:
+                  menu_driver(menu, REQ_UP_ITEM);
+                  break;
+              case ACTION_NPAGE:
+                  menu_driver(menu, REQ_SCR_DPAGE);
+                  break;
+              case ACTION_PPAGE:
+                  menu_driver(menu, REQ_SCR_UPAGE);
+                  break;
+              case ACTION_CONFIRM:
+              case ACTION_SELECT:
+                  // Change sort attribute
+                  sort = sip_sort_options();
+                  id = sip_attr_from_name(item_name(current_item(info->menu)));
+                  if (sort.by == id) {
+                      sort.asc = (sort.asc) ? false : true;
+                  } else {
+                      sort.by = id;
+                  }
+                  sip_set_sort_options(sort);
+                  /* no break */
+              case ACTION_PREV_SCREEN:
+                  // Desactive sorting menu
+                  info->menu_active = 0;
+
+                  // Restore list position
+                  mvderwin(info->list_win, 4, 0);
+                  // Restore list window size
+                  wresize(info->list_win, ui->height - 5, ui->width);
+
+                  // Remove menu and items
+                  unpost_menu(info->menu);
+                  free_menu(info->menu);
+                  break;
+              default:
+                  // Parse next action
+                  continue;
+          }
+
+          // We've handled this key, stop checking actions
+          break;
+      }
+
+      // Return if this panel has handled or not the key
+      return (action == ERR) ? KEY_NOT_HANDLED : KEY_HANDLED;
+}
+
 int
 call_list_help(ui_t *ui)
 {
@@ -922,3 +989,39 @@ call_list_move(ui_t *ui, int line)
         }
     }
 }
+
+void
+call_list_select_sort_attribute(ui_t *ui)
+{
+    call_list_info_t *info;
+    ITEM *items[SIP_ATTR_COUNT + 1];
+    int i;
+
+    // Get panel info
+    if (!(info = call_list_info(ui)))
+        return;
+
+    // Activete sorting menu
+    info->menu_active = 1;
+
+    wresize(info->list_win, ui->height - 5, ui->width - 12);
+    mvderwin(info->list_win, 4, 12);
+
+    // Create menu entries
+    for (i = 0; i < info->columncnt; i++) {
+        items[i] = new_item(sip_attr_get_name(info->columns[i].id), 0);
+    }
+    items[info->columncnt] = NULL;
+    // Create the columns menu and post it
+    info->menu = new_menu(items);
+
+    // Set main window and sub window
+    set_menu_win(info->menu, ui->win);
+    set_menu_sub(info->menu, derwin(ui->win, 20, 15, 4, 0));
+    werase(menu_win(info->menu));
+    set_menu_format(info->menu, ui->height, 1);
+    set_menu_mark(info->menu, "");
+    set_menu_fore(info->menu, COLOR_PAIR(CP_DEF_ON_BLUE));
+    menu_opts_off(info->menu, O_ONEVALUE);
+    post_menu(info->menu);
+}

src/curses/ui_call_list.h

@@ -33,6 +33,7 @@
 #define __UI_CALL_LIST_H
 
 #include "ui_manager.h"
+#include <menu.h>
 #include "scrollbar.h"
 
 /**
@@ -89,6 +90,10 @@ struct call_list_info {
     FIELD *fields[FLD_LIST_COUNT + 1];
     //! We're entering keys on form
     int form_active;
+    // Columns menu
+    MENU *menu;
+    //! We're selecting sorting field
+    int menu_active;
     //! Move to last list entry if autoscroll is enabled
     int autoscroll;
     //! List scrollbar
@@ -244,11 +249,23 @@ call_list_handle_key(ui_t *ui, int key);
  *
  * @param ui UI structure pointer
  * @param key Pressed keycode
-* @return enum @key_handler_ret
+ * @return enum @key_handler_ret
  */
 int
 call_list_handle_form_key(ui_t *ui, int key);
 
+/**
+ * @brief Handle Sort menu key strokes
+ *
+ * This function will manage the custom keybidnings for sort menu
+ *
+ * @param ui UI structure pointer
+ * @param key Pressed keycode
+ * @return enum @key_handler_ret
+ */
+int
+call_list_handle_menu_key(ui_t *ui, int key);
+
 /**
  * @brief Request the panel to show its help
  *
@@ -299,4 +316,15 @@ call_list_clear(ui_t *ui);
 void
 call_list_move(ui_t *ui, int line);
 
+/**
+ * @brief Select column to sort by
+ *
+ * This function will display a lateral menu to select the column to sort
+ * the list for. The menu will be filled with current displayed columns.
+ *
+ * @param ui UI structure pointer
+ */
+void
+call_list_select_sort_attribute(ui_t *ui);
+
 #endif

src/sip_call.c

@@ -292,13 +292,22 @@ call_attr_compare(sip_call_t *one, sip_call_t *two, enum sip_attr_id id)
             break;
         default:
             // Get attribute values
+            memset(onevalue, 0, sizeof(onevalue));
+            memset(twovalue, 0, sizeof(twovalue));
             call_get_attribute(one, id, onevalue);
             call_get_attribute(two, id, twovalue);
             comparetype = 0;
+            break;
     }
 
     switch (comparetype) {
         case 0:
+            if (strlen(twovalue) == 0 && strlen(onevalue) == 0)
+                return 0;
+            if (strlen(twovalue) == 0)
+                return 1;
+            if (strlen(onevalue) == 0)
+                return -1;
             return strcmp(onevalue, twovalue);
         case 1:
             if (oneintvalue == twointvalue) return 0;