kentreez dev

Wick Ref: http://codeigniter.com/wiki/Wick/
Matchbox Ref: http://code.google.com/p/matchbox/

ผมพบว่าใน php4 จะมีการพยายามโหลดคลาส Wick สองครั้งครับ
(ผมตั้ง config ให้ autoload libraries Wick)

จริงๆ แล้ว Zacharias Knudsen เค้าบอกว่า Wick มัน php4 compatible นะครับ. ทำให้ผมไม่แน่ใจว่าที่ผมเจอ Error Redeclare นี่เพราะผมแก้ Framework ไปหลายจุดหรือป่าว?

แต่ไม่มีปัญหาครับ Error Redeclare เรื่องเล็กนิดเดียวครับ ก็แค่ดัก If เข้าไปนิดหน่อยก็เรียบร้อย แต่ไม่รู้ว่าจะมีผลต่อ Performance หรือป่าว และ Reliable ?. ในส่วนของ Performance ก็ไม่รู้จะทดสอบอย่างไรเหมือนกัน แต่ผมได้ทำการทดลองใช้งาน $this->load->uri ทั้งใน App และใน Module ซึ่งยังสามารถทำงานได้ปรกติ ก็ถือว่าโอเคครับ (มั้งนะ)

ถ้ามีเวลาเหลือ ผมคงจะหาวิธีแก้ที่สาเหตุ มันจริงๆ อีกทีครับ 🙂

แก้ไขแบบง่ายๆ ครับ  (เติมโค๊ด ๓ ตำแหน่ง)

จุดแรก

[php]if (class_exists('Wick')===FALSE) { // <-- Insert this line (at line 36)[/php]

จุดที่สอง

[php]function Wick ()
{
<span style="color: #0000ff;">static $php4_run_once; // <– Insert this line</span>
<span style="color: #0000ff;">if (isset($php4_run_once)) // <– Insert this line
return; // <– Insert this line
else // <– Insert this line
$php4_run_once = true; // <– Insert this line</span>
$ci = &get_instance();[/php]

จุดที่สาม

[php]} // <-- Insert this line</span>

/* End of file Wick.php */

/* Location: ./system/application/libraries/Wick.php */[/php]

	

รักสามเศร้าในไฟล์ /system/codeigniter/Common.php

ในฟังก์ชัน load_class ตรงส่วนที่เป็นการตัดสินใจว่าจะดึง library จากใน BASEPATH หรือ APPPATH มีโค๊ดตามนี้

if (file_exists(APPPATH.'libraries/'.config_item('subclass_prefix').$class.EXT))
{
    require(BASEPATH.'libraries/'.$class.EXT);
    require(APPPATH.'libraries/'.config_item('subclass_prefix').$class.EXT);
    $is_subclass = TRUE;
}
else
{
    if (file_exists(APPPATH.'libraries/'.$class.EXT))
    {
        require(APPPATH.'libraries/'.$class.EXT);
        $is_subclass = FALSE;
    }
    else
    {
        require(BASEPATH.'libraries/'.$class.EXT);
        $is_subclass = FALSE;
    }
}

จะเห็นว่าถ้ามี MY_Loader.php อยู่ใน /system/application/libraries/ ก็จะเข้าที่เงื่อนไขแรกเลย. ซึ่งมันจะโหลด Loader.php ซึ่งอยู่ใน /system/libraries/ โดยไม่สนว่าจะมี Loader.php อยู่ใน /system/application/libraries/ หรือไม่ ทั้งๆ ที่จริงๆ แล้วควรให้ความสำคัญกับ Loader ที่อยู่ใน APPPATH ก่อน BASEPATH. ตรงนี้ผมก็ไม่แน่ใจเหมือนกันว่าจุดประสงค์ที่แท้จริงของเงื่อนไขนี้คืออะไร แต่มันไม่ตรงกับความต้องการของผม ก็เลยต้องจัดการปรับเปลี่ยนเล็กน้อย เป็นแบบนี้แทนครับ

if (file_exists(APPPATH.'libraries/'.config_item('subclass_prefix').$class.EXT))
{
    if (file_exists(APPPATH.'libraries/'.$class.EXT))
        require(APPPATH.'libraries/'.$class.EXT);
    else
        require(BASEPATH.'libraries/'.$class.EXT);
    require(BASEPATH.'libraries/'.$class.EXT);
    require(APPPATH.'libraries/'.config_item('subclass_prefix').$class.EXT);
    $is_subclass = TRUE;
}
else
{
    if (file_exists(APPPATH.'libraries/'.$class.EXT))
    {
        require(APPPATH.'libraries/'.$class.EXT);
        $is_subclass = FALSE;
    }
    else
    {
        require(BASEPATH.'libraries/'.$class.EXT);
        $is_subclass = FALSE;
    }
}