[asterisk-bugs] [Asterisk-GUI 0013748]: [patch] 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-21 14:52 CDT
====================================================================== 
Summary:                    [patch] 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.
====================================================================== 

---------------------------------------------------------------------- 
 (0094078) bkruse (manager) - 2008-10-21 14:52
 http://bugs.digium.com/view.php?id=13748#c94078 
---------------------------------------------------------------------- 
This patch is great dkerr, however, I will have to take out your comments
/* DAK */ because we do not allow that in the source. If you contribute
enough, I can of course mention you in the contributors file, and you will
be mentioned in the commit message.

(It is like this for Asterisk as well, against the coding guidelines)

-bk 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-10-21 14:52 bkruse         Note Added: 0094078                          
======================================================================