Open Source Training Seminar FreePBX Paid Support

Ticket #1285 (closed Bugs: fixed)

Opened 2 years ago

Last modified 2 years ago

Use of HTTP_SESSION_VARS in CDR scripts breaks in PHP 5

Reported by: gsiemon Assigned to: RobThomas
Priority: trivial Milestone: 2.2
Component: Web interface Version: 2.2beta2
Keywords: CDR Cc:
Confirmation: SVN Revision (if applicable):
Backend Engine: All Backend Engine Version:

Description

The following CDR files are using HTTP_SESSION_VARS in the code.

call-comp.php call-daily-load.php call-last-month.php graph_hourdetail.php graph_pie.php graph_statbar.php graph_stat.php

This AFAIK has been depricated in favour of _SESSION. I have PHP 5.04 installed on my Centos 4.4 system and find that none of the CDR functions work unless I manually change the HTTP_SESSION_VARS to _SESSION.

This appears to have been reported before in #281 but the ticket was closed out automatically for some reason.

Attached files work on PHP 5.04.

Attachments

CDR fixes.zip (30.3 kB) - added by gsiemon on 11/07/06 02:15:02.
Patched files to fix problem

Change History

11/07/06 02:15:02 changed by gsiemon

  • attachment CDR fixes.zip added.

Patched files to fix problem

11/07/06 21:19:00 changed by RobThomas

You seem to have attached the _un_ modified files.. Yours: 

[root@wpm4l-gw cdr]# md5sum call-comp.php   
e4020bf410c8f9c249ae7f9b2086170f  call-comp.php

Standard 

[root@wpm4l-gw cdr]# md5sum call-comp.php
e4020bf410c8f9c249ae7f9b2086170f  call-comp.php

--Rob

11/07/06 21:22:16 changed by RobThomas

  • owner set to RobThomas.
  • status changed from new to assigned.

..However, if that's the only change, (s/HTTP_SESSION_VARS/_SESSION/), then you might want to check r3029 to make sure that's what should be done. Thanks!

--Rob

11/08/06 03:55:53 changed by gsiemon

Oops! :-( Those were the only changes. Thanks.

Greg

11/08/06 15:08:24 changed by RobThomas

  • status changed from assigned to closed.
  • resolution set to fixed.

01/08/07 11:46:44 changed by

  • milestone deleted.

Milestone 2.2 deleted

01/08/07 12:47:40 changed by vgster

  • milestone set to 2.2.
Donate



Support
Download
Develop
Forums
News
Documentation
Paid Support
About

Paid Ads