Changeset 11925

Show
Ignore:
Timestamp:
08/30/07 23:16:19 (1 year ago)
Author:
rambo
Message:

refs #102 check that the link_value is not empty before trying to use it

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/midcom/midcom.core/midcom/services/dbclassloader.php

    r11923 r11925  
    830830            list (\$link_value, \$dummy) = each(\$link_values); 
    831831            unset(\$mc, \$link_values, \$dummy); 
    832             \$mc2 = new midgard_collector('{$this->_class_definition['new_class_name']}', '{$target_property}', \$link_value); 
    833             \$mc2->set_key_property('guid'); 
    834             \$mc2->execute(); 
    835             \$guids = \$mc2->list_keys(); 
    836             if (!is_array(\$guids)) 
    837             { 
    838                 unset(\$mc2, \$guids, \$link_value); 
    839                 return null; 
    840             } 
    841             list (\$parent_guid, \$dummy) = each(\$guids); 
    842             unset(\$mc2, \$guids, \$link_value, \$dummy); 
    843             return \$parent_guid; 
     832            if (!empty(\$link_value)) 
     833            { 
     834                \$mc2 = new midgard_collector('{$this->_class_definition['new_class_name']}', '{$target_property}', \$link_value); 
     835                \$mc2->set_key_property('guid'); 
     836                \$mc2->execute(); 
     837                \$guids = \$mc2->list_keys(); 
     838                if (!is_array(\$guids)) 
     839                { 
     840                    unset(\$mc2, \$guids, \$link_value); 
     841                    return null; 
     842                } 
     843                list (\$parent_guid, \$dummy) = each(\$guids); 
     844                unset(\$mc2, \$guids, \$link_value, \$dummy); 
     845                return \$parent_guid; 
     846            } 
     847            else 
     848            { 
     849                unset(\$mc2, \$guids, \$link_value, \$dummy); 
     850            } 
    844851        } 
    845852        else 
    …… 
    865872            list (\$link_value, \$dummy) = each(\$link_values); 
    866873            unset(\$mc, \$link_values, \$dummy); 
    867             \$mc2 = new midgard_collector('{$target_class}', '{$target_property}', \$link_value); 
    868             \$mc2->set_key_property('guid'); 
    869             \$mc2->execute(); 
    870             \$guids = \$mc2->list_keys(); 
    871             if (!is_array(\$guids)) 
    872             { 
    873                 unset(\$mc2, \$guids, \$link_value); 
    874                 return null; 
    875             } 
    876             list (\$parent_guid, \$dummy) = each(\$guids); 
    877             unset(\$mc2, \$guids, \$link_value, \$dummy); 
    878             return \$parent_guid; 
     874            if (!empty(\$link_value)) 
     875            { 
     876                \$mc2 = new midgard_collector('{$target_class}', '{$target_property}', \$link_value); 
     877                \$mc2->set_key_property('guid'); 
     878                \$mc2->execute(); 
     879                \$guids = \$mc2->list_keys(); 
     880                if (!is_array(\$guids)) 
     881                { 
     882                    unset(\$mc2, \$guids, \$link_value); 
     883                    return null; 
     884                } 
     885                list (\$parent_guid, \$dummy) = each(\$guids); 
     886                unset(\$mc2, \$guids, \$link_value, \$dummy); 
     887                return \$parent_guid; 
     888            } 
     889            else 
     890            { 
     891                unset(\$mc2, \$guids, \$link_value, \$dummy); 
     892            } 
    879893        } 
    880894        else