Zend Framework

BC Break in Zend_Log_Writer_ZendMonitor

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.10.7, 1.10.8, 1.11.0
  • Fix Version/s: 1.11.1
  • Component/s: Zend_Log
  • Labels:
    None

Description

After holiday I upgraded to Zend Framework 1.10.7 and made my first bug and found out that Zend_Log_Writer_ZendMonitor was failing using Zend Server 5.0.2.

After doing some research i found this bug http://framework.zend.com/issues/browse/ZF-10081 that is been fixed for 1.10.7. User who filled out that bug was using Zend Platform (product that has come discontinued).

Zend_Log_Writer_ZendMonitor is ment to use with Zend Server Monitor, says Reference Guide so I did some digging and found that api for monitor has changed from Platform to Server:

from: void monitor_custom_event(string $class, string $text[, integer $severe, mixed $user_data])
to: void zend_monitor_custom_event (string $class, string $text, mixed $user_data)

Some how monitor_custom_event-function is still available for Zend Server, but singnature is same as it is for zend_monitor_custom_event.

Both Platform and Server can be supported (I think, but didn't test) doing sniff if zend_monitor_custom_event (I think this isn't available in Zend Platform) is present then using it and if not using monitor_custom_event.

Or __constructor should fail if Zend Platform is used.

Issue Links

Activity

Hide
Margus Sipria added a comment -

patch for my local mirror

Show
Margus Sipria added a comment - patch for my local mirror
Hide
Margus Sipria added a comment -

Assigning to person who broke this first place

Show
Margus Sipria added a comment - Assigning to person who broke this first place
Hide
Matthew Weier O'Phinney added a comment -

Fixed in trunk and 1.11 release branch. The original code was based off of a beta or RC of Zend Server – and the API was later changed. Additionally, it was modified such that it was no longer compatible with Zend Platform.

I've updated the code to "sniff" for Zend Server, and to operate differently based on whether Zend Server is detected or Zend Platform is used.

The fix will be in 1.11.1.

Show
Matthew Weier O'Phinney added a comment - Fixed in trunk and 1.11 release branch. The original code was based off of a beta or RC of Zend Server – and the API was later changed. Additionally, it was modified such that it was no longer compatible with Zend Platform. I've updated the code to "sniff" for Zend Server, and to operate differently based on whether Zend Server is detected or Zend Platform is used. The fix will be in 1.11.1.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: