[asterisk-bugs] [Asterisk-GUI 0013748]: Problems with Call Detail Report beta page

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13748 
====================================================================== 
Reported By:                dkerr
Assigned To:                bkruse
====================================================================== 
Project:                    Asterisk-GUI
Issue ID:                   13748
Category:                   General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     assigned
Asterisk Version:           1.4.21.2 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 3966 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-10-20 13:22 CDT
Last Modified:              2008-10-20 14:48 CDT
====================================================================== 
Summary:                    Problems with Call Detail Report beta page
Description: 
Found several problems with beta version of CDR report in svn 3966.

1) First time I used it, I got http 404 Not Found instead of my CDR list.
On debugging I found that the code is dependent upon finding a "prefix="
statement in http.conf. My (presumably default, I've never changed it)
http.conf file did not have this statement. When I added "prefix=asterisk"
that solved the problem. BUT why is this needed? No other page in the GUI
requires it?

2) page states that list is "most recent first". It is not, most recent is
last (fix should be to re-sort the list, not change the statement!)

3) I think that the column headings should be fixed, not scrollable. In
other words, the frame that contains the CDR list should only have the CDR
list. The column headings should be outside of the frame.
====================================================================== 

---------------------------------------------------------------------- 
 (0094000) dkerr (reporter) - 2008-10-20 14:48
 http://bugs.digium.com/view.php?id=13748#c94000 
---------------------------------------------------------------------- 
I'll work on a patch for the sort order when I get time this week.

I thought that sysinfo would do the same, which is why I was surpised that
it didn't work, when sysinfo does.  So I looked at the source to sysinfo
and found that it actually calls a system commend (like ifconfig) and pipes
the output, which is implemented in astman.js. In here it uses different
logic to determine the target directory that is not dependent upon
http.conf, rather it uses ASTGUI.paths['guiInstall'] which turns out to be
hardcoded in astman.js.  Maybe not good practice to be hardcoded, but that
is how it is.

Whatever the method, the GUI should be constent.


And finally... yes you need the scrollbar. But the column headings should
not scroll, only the column data. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-10-20 14:48 dkerr          Note Added: 0094000                          
======================================================================