PHP Class Help

[brsy]

I have a template system that works perfectly. It's just that I wanted to be able to organize the files, so it would be much more neat. This code is throwing errors:

public function getFile($file, $admin = 0) {
       
        global $pelican;
       
        if($admin = 1) {
            if(file_exists('_templates/'.$pelican['Site']['skin'].'/admin' . $file . '.tpl')) {
                ob_start();
                include('_templates/'.$pelican['Site']['skin'].'/admin' . $file . '.tpl');
                $this->tpl .= ob_get_contents();
                ob_end_clean();
            }
           
        }
       
        if(file_exists('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl')) {
            ob_start();
            include('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl');
            $this->tpl .= ob_get_contents();
            ob_end_clean();
        }
                else {
                    die("<br /><b>Templating Error:</b> Unable to get the file <b>'".$file."'</b> from the theme <b>".$pelican['Site']['skin']."</b>.");
                }
            }

The original code is:

 if(file_exists('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl')) {
            ob_start();
            include('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl');
            $this->tpl .= ob_get_contents();
            ob_end_clean();
        }
                else {
                    die("<b>Templating Error:</b> Unable to get the file <b>'".$file."'</b> from the theme <b>".$pelican['Site']['skin']."</b>.");
                }
            }

 

[Heaplink]

What errors does it throw?

 

[brsy]

I made it die, as you can see in the else statement. It died like this:

Templating Error: Unable to get the file 'index' from the theme Bootstrap.

 

[Heaplink]

Make sure the index file is located in the right place, and it the .tpl extension.

 

[brsy]

It is, indeed.

You must be registered for see links

 

[Heaplink]

The admin/index.php is not a TPL, this is what might cause the error if you have set up the rank.

 

[brsy]

Look at _templates->Bootstrap->Admin->Index.tpl

 

[Heaplink]

YOu should know where the script executes from when you enter the address. Look in your .htaccess and see if you see any redirects.

 

[brsy]

YOu should know where the script executes from when you enter the address. Look in your .htaccess and see if you see any redirects.

Didn't use any .htaccess
I am still trying to get used to Mac, and I have everything working properly. Plus, all I did was add if($admin = 1) { (same code, except get the .tpl from admin file) }

 

[Heaplink]

The error is not thrown at the admin check, since you don't make an else there. It's thrown in the last if else check.

 

[Kryptos]

Run this code, see what throws you then you'll find the error.

echo '_templates/'.$pelican['Site']['skin'].'/admin' . $file . '.tpl'; die;

Probably $pelican['Site']['skin'] returns null

OR

it needs to be: '_templates/'.$pelican['Site']['skin'].'/admin/' . $file . '.tpl'

 

[Heaplink]

Yeah Kryptos is right, the last slash must be it. The pelican var is not throwing null in the die, so it must be the last one.

 

[brsy]

Okay, so it isn't throwing an error anymore; but now it's freaking only adding the footer, it isn't getting the stupid file.

Actually, it only works for one thing -- whether it's the main site, or the admin file directory. This is the current code that I have at the moment.

public function getFile($file, $admin = null) {
     
        global $pelican;
     
     
        if($admin == 1) {
            if(file_exists('_templates/'.$pelican['Site']['skin'].'/admin/' . $file . '.tpl')) {
                ob_start();
                include('_templates/'.$pelican['Site']['skin'].'/admin/' . $file . '.tpl');
                $this->tpl .= ob_get_contents();
                ob_end_clean();
            }
        }
     
        if($admin == null) {
            if(file_exists('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl')) {
                ob_start();
                include('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl');
                $this->tpl .= ob_get_contents();
                ob_end_clean();
            }
        }
                else {
                    die("<br /><b>Templating Error:</b> Unable to get the file <b>'".$file."'</b> from the theme <b>".$pelican['Site']['skin']."</b>.");
                }
            }

If I put the if($admin == 1) part first, the Admin section doesn't work, and if I put if($admin == null) then the admin part works.

 

[Kryptos]

*sigh* lrn2phpd00d

public function getFile($file, $admin = null) {
   
        global $pelican;
   
   
        if($admin == 1) {
            if(file_exists('_templates/'.$pelican['Site']['skin'].'/admin/' . $file . '.tpl')) {
                ob_start();
                include('_templates/'.$pelican['Site']['skin'].'/admin/' . $file . '.tpl');
                $this->tpl .= ob_get_contents();
                ob_end_clean();
            }
        }else{
            if(file_exists('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl')) {
                ob_start();
                include('_templates/'.$pelican['Site']['skin'].'/' . $file . '.tpl');
                $this->tpl .= ob_get_contents();
                ob_end_clean();
            }else{
                    die("<br /><b>Templating Error:</b> Unable to get the file <b>'".$file."'</b> from the theme <b>".$pelican['Site']['skin']."</b>.");
            }
        }
            
 }

 

[brsy]

Ugh, no errors, but it still dnt work

You must be registered for see links

. I have my files kinda like uberCMS, and i have it to add sets, so I would add footer, etc. Only the footer and header is added, meaning the getFile thingy is ignored.

 

[Kryptos]

Ugh, no errors, but it still dnt work

You must be registered for see links

. I have my files kinda like uberCMS, and i have it to add sets, so I would add footer, etc. Only the footer and header is added, meaning the getFile thingy is ignored.

Probably you're not outputting $this->tpl right or something. That function is perfect.