bugGNU FM - Bugs: bug #26395, Created logging mechanism for...

 
 

bug #26395: Created logging mechanism for nixtape

Submitter:  Not Available <piotrszulawski>
Submitted:  Wed 29 Apr 2009 08:34:18 AM UTC
   
 
Category:  nixtape/alpha Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 08 May 2009 10:47:28 PM UTC, comment #5: 

We already have a logging mechanism with the reportError() function. This needs to be more widely used, but that's a separate issue, so I'm closing this bug.

Michael Sheldon <elleo>
Group Member
Wed 06 May 2009 07:38:05 PM UTC, comment #4: 

please review

Not Available <piotrszulawski>
Sat 02 May 2009 12:07:02 PM UTC, comment #3: 

Toby:

I assume `set_error_handler' and `trigger_error' are quite old-fashioned methods of error reporting since PHP provides an exception model and libre.fm is supposed to be written in an object-oriented manner (I hope so, at least).

Also, using these functions for logging seems a bit weird to me -- have to trigger an error in order to log a debug message, huh?

Hence I believe Piotr's class or a similar logging mechanism will be used in libre.fm.

-Deleted Account- <qwp0>
Wed 29 Apr 2009 11:38:30 AM UTC, comment #2: 

Okay - i just reused my old class.

If you don't want to use it - then don't -  but afair there isn't any serious logging mechanism used in nixtape.

Not Available <piotrszulawski>
Wed 29 Apr 2009 11:09:51 AM UTC, comment #1: 

Why is this needed? PHP already has pretty good built-in logging. set_error_handler + trigger_error are capable of a lot.


Toby Inkster <tobyink>
Group Member
Wed 29 Apr 2009 08:34:18 AM UTC, original submission:  

Attached Logger.php

My class used for debugging. It writes messages to file. Can be extended.

Not Available <piotrszulawski>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #18051:  Logger.php added by piotrszulawski (3KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by elleo (Posted a comment)
  • -email is unavailable- added by qwp0 (Posted a comment)
  • -email is unavailable- added by tobyink (Posted a comment)
  • -email is unavailable- added by piotrszulawski (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-05-08 elleo StatusNone Fixed
        Open/ClosedOpen Closed
    2009-04-29 piotrszulawski Attached File- Added Logger.php, #18051

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code