Opened 5 years ago

Closed 5 years ago

#1484724 closed Feature Patches (invalid)

pagesize 0 default, preview pane, sqlite

Reported by: jaimeco Owned by:
Priority: 5 Milestone:
Component: Interface Version: 0.1-rc2
Severity: normal Keywords: pagesize preview pan vista preliminar Filas por pagina
Cc:

Description (last modified by thomasb)

On version 0.1-rc2 the pagesize is always 0, even the user changed by another value, always is set 0 in sqlite table: users column preferences.
For example if you can't view any message then you save any preference, you can modify sqlite just like that:

sqlite>  select * from users where user_id="31";
31|jaimeco|mail.usac.edu.gt||2008-01-25 10:33:42|2008-01-25 11:06:44|en_GB|a:10:{s:8:"timezone";d:-6;s:10:"dst_active";b:0;s:8:"pagesize";i:60;s:0:
"prettydate";b:1;s:11:"prefer_html";b:1;s:10:"htmleditor";b:0;s:12:"preview_pane";b:1;s:14:
"draft_autosave";i:180;s:16:"message_sort_col";s:4:"date";s:18:"message_sort_order";s:4:"DESC";}

pay attention on the last field: preferences have setted with: "pagesize";i:60;s:0

then, if you chaged that value bolded by another different of zero you can view messages now.

sqlite> update users set preferences='a:10:{s:8:"timezone";d:-6;s:10:"dst_active";b:0;s:8:"pagesize";i:5;s:10:
"prettydate";b:1;s:11:"prefer_html";b:1;s:10:"htmleditor";b:0;s:12:"preview_pane";b:1;s:14:
"draft_autosave";i:600;s:16:"message_sort_col";s:4:"date";s:18:"message_sort_order";s:4:"DESC";}' where user_id="31";

The problem has a solve if you modify program/steps/settings/save_prefs.inc

//if (isset($CONFIG['max_pagesize']) && ($a_user_prefs['pagesize'] > $CONFIG['max_pagesize']))
if (!isset($CONFIG['max_pagesize']) && ($a_user_prefs['pagesize'] > $CONFIG['max_pagesize']))
  {
  $a_user_prefs['pagesize'] = (int) $CONFIG['max_pagesize'];
  }

Good look, it works now.


On version 0.1-rc2 the preview pane works if you replace in program/steps/settings/func.inc

// show config parameter for preview pane

//if (!isset($no_override['preview_pane']))
  if (isset($no_override['preview_pane']))

That modify permit show the option "Show preview pane" in Preferences menu.


--jaimeco:(|)

Change History (3)

comment:1 Changed 5 years ago by thomasb

  • Description modified (diff)
  • Severity changed from major to normal

(Better formatting of report text)

comment:2 Changed 5 years ago by thomasb

I cannot confirm these problems. The current code works correctly.

comment:3 Changed 5 years ago by thomasb

  • Resolution set to invalid
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.