Changeset 17765

Show
Ignore:
Timestamp:
09/29/08 16:13:40 (2 months ago)
Author:
rambo
Message:

better logging and error reporting

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • branches/MidCOM_2_8/midcom.helper.replicator/exporter.php

    r16794 r17765  
    110110            { 
    111111                unset($privileges, $privilege, $qb); 
    112                 debug_add("midcom_helper_replicator_serialize returned false for privilege {$privilege->guid}", MIDCOM_LOG_ERROR); 
     112                debug_add("midcom_helper_replicator_serialize returned false for privilege {$privilege->guid}, errstr: " . mgd_errstr(), MIDCOM_LOG_ERROR); 
    113113                debug_pop(); 
    114114                return false; 
    …… 
    140140        if ($attachment_serialized === false) 
    141141        { 
    142             debug_add("midcom_helper_replicator_serialize returned false for attachment {$attachment->guid}", MIDCOM_LOG_ERROR); 
     142            debug_add("midcom_helper_replicator_serialize returned false for attachment {$attachment->guid}, errstr: " . mgd_errstr(), MIDCOM_LOG_ERROR); 
    143143            debug_pop(); 
    144144            return false; 
    …… 
    150150        { 
    151151            unset($attachment_serialized); 
    152             debug_add("midcom_helper_replicator_serialize_blob returned false for attachment {$attachment->guid}", MIDCOM_LOG_ERROR); 
     152            debug_add("midcom_helper_replicator_serialize_blob returned false for attachment {$attachment->guid}, errstr: " . mgd_errstr(), MIDCOM_LOG_ERROR); 
    153153            debug_pop(); 
    154154            return false; 
    …… 
    275275            { 
    276276                unset($parameters, $parameter, $qb); 
    277                 debug_add("midcom_helper_replicator_serialize returned false for parameter {$parameter->guid}", MIDCOM_LOG_ERROR); 
     277                debug_add("midcom_helper_replicator_serialize returned false for parameter {$parameter->guid}, errstr: " . mgd_errstr(), MIDCOM_LOG_ERROR); 
    278278                debug_pop(); 
    279279                return false; 
    …… 
    370370        { 
    371371             $GLOBALS['midcom_helper_replicator_logger']->pop_prefix(); 
    372             debug_add("midcom_helper_replicator_serialize returned false for object {$object->guid}", MIDCOM_LOG_ERROR); 
     372            debug_add("midcom_helper_replicator_serialize returned false for object {$object->guid}, errstr: " . mgd_errstr(), MIDCOM_LOG_ERROR); 
    373373            debug_pop(); 
    374374            return false; 

  • branches/MidCOM_2_8/midcom.helper.replicator/helpers.php

    r16788 r17765  
    2929        || $object->lang === 0) 
    3030    { 
    31         // Non-ML or not in langx does not trigger the bug 
     31        // Non-ML or not in langx does not trigger bug #259 
    3232        $stat = midgard_replicator::serialize($object); 
    3333        return midcom_helper_replicator_serialize_check_bug244($stat, $object); 
    …… 
    105105        $_MIDCOM->uimessages->add('midcom.helper.replicator', $msg, 'error'); 
    106106        $GLOBALS['midcom_helper_replicator_logger']->log_object($object, "has {$total_langs} languages, this triggers bug #244, preventing export.", MIDCOM_LOG_ERROR); 
     107        debug_push_class('function', __FUNCTION__); 
     108        debug_add("Object {$class} #{$object->id} has {$total_langs} languages, this triggers bug #244, preventing export", MIDCOM_LOG_WARN); 
     109        debug_pop(); 
    107110        unset($class, $object_url, $msg, $total_langs, $langs); 
    108         //debug_pop(); 
     111        mgd_set_errno(MGD_ERR_ERROR); 
    109112        return false; 
    110113    }