CodeIgniter Forums

$this->config->item( … ) default values

2008-08-27T18:21:48-06:00

When I try to get the configuration for something like


$this->config->item('cache_path');

I get an empty string unless it has been explicitly set in the config.php file. I think it would be more convenient if the default config value was returned. (In this case, ‘system/cache/’ .)

Add checks for PHP version 6 before magic_quotes

2008-08-09T16:28:54-06:00

PHP 6 lacks the magic_quotes methods, so here are the changes I’ve made to my CI install to make it run flawlessly on PHP 6:

codeigniter/CodeIgniter.php, line 60:

set_magic_quotes_runtime(0); // Kill magic quotes

// to

if(floor(phpversion()) < 6)
{
    set_magic_quotes_runtime(0); // Kill magic quotes
}

libraries/Input.php, line 173:

if (get_magic_quotes_gpc())
{

// to
if (floor(phpversion()) < 6 && get_magic_quotes_gpc())
{

It would be nice if this small change can be added to CodeIgniter, just to simplify when testing on PHP6 (and for the future ).

tabindex on forum login form

2008-09-05T03:40:44-06:00

There’s no tabindex on the little login form on the forums. It’s no that big of a deal, but it would be handy!

Also, a tabindex on the title and message fields when posting a new topic would be cool. It would make a lazy tabbing man’s life easier!

REST & SOAP Libraries

2008-09-04T17:19:08-06:00

Please add core libraries for REST and SOAP. I use the XML-RPC library all the time and love it, only problem I run into is when I try to integrate with other Web APIs they are almost always REST or SOAP.

AR library : alternative to caching sql statement building methods

2008-09-04T03:55:36-06:00

I just something like this

if(something)
         {
             $id = $this->db->select('id')->where($something)->from('table')->get()->row()->id;

             $this->db->where($something);
         }
         else
         {
             $id = $this->uri->segment(3);
             $this->db->where('id',$id);
         }
         $this->db->from('table');
         $this->db->delete();

         // ...

This because i couldn’t use caching on the where and from method parameters at the same time. So i started thinking what if i could do

$this->db->from('table');
         if(something)
         {
             $id = $this->db->select('id')->where($something)->get()->row()->id;
         }
         else
         {
             $id = $this->uri->segment(3);
             $this->db->where('id',$id);
         }
         $this->db->delete();

         // ...

The select would be disregarded but the where data would be used by the delete method. I browsed through the AR code and i found out that the delete method has a delete_data flag. Maybe this should be possible for all methods that reset the sql statement data.

The simplest solution is to add a


var reset_data = TRUE;

class variable. Then my ‘perfect’ code will be

$this->db->from('table');
         if(something)
         {
              $this->db->reset_data = FALSE;
              $id = $this->db->select('id')->where($something)->get()->row()->id;
              $this->db->reset_data = TRUE;
         }
         else
         {
             $id = $this->uri->segment(3);
             $this->db->where('id',$id);
         }
         $this->db->delete();

         // ...

And in the methods all you need to do to make this work for all those resetting methods is

if($this->reset_data)
{
     $this->_reset_select();
}
// or
if($this->reset_data)
{
     $this->_reset_write();
}

This is also a solution that is backward compatible.

FirePHP?

2008-08-30T16:08:41-06:00

This may just be me, but I’ve find a nice little add on for FireBug called FirePHP. People may’ve heard of it and may use it, but for those who haven’t, here’s the link: http://www.firephp.org/

It allows you to echo out debug messages without affecting the body of the page as it’s done in the headers.

It’ll be cool if this is added as a plugin to codeignitor as it’s really handy.

HTML helper doctype function

2008-09-01T09:29:21-06:00

Hello everyone at Codeigniter forums, I’m new here.

All I wanted to say that it would be nice to have a helper function in the HTML helper that would generate the doctype. I know, it may sound silly, but no one learns it by heart, and every time you start a new html file that needs a doctype, you go to a website to copy the doctype. Wouldn’t it be easier by just typing


<?=doctype('xhtml-strict')?>

or something like that?
I know the function is a piece of cake and you might say: “why don’t you do your own?”. Of course I can do it, but why wouldn’t it benefit all of us?

Multiple DB Support - Session Class Request (attached)

2008-09-02T15:19:48-06:00

@Rick & Dereks—Please consider this change to the Session class as a supplement to the other improvements you’re making. Allow the Session data to be stored in a DB based upon an additional config item setting that identifies the database group that will receive the the session data.

New config entry: $config[’sess_database_group’] = ‘db_group’;

A simple change really to go along with the file changes attached.

RATIONALE: There are a couple of primary reasons and a dozen or so that don’t really matter. The two reasons that make the biggest difference:

1) Convention is that user credentials be stored independently of application data. The DB query stuff is hard coded in the session class so that it can only be stored in the ‘default’ or user loaded DB if they specify the group at load time. The convention would have the authentication session stored with the user authentication data as an option for the developer at a minimum.

2) CI supports multiple DBs, so should the API at the core.

------

You’ll see the changes are quite simple. I’ve set them off with -> at column 0 of the file so your syntax highter should have no trouble finding the changes . Yes, I’ve pretty much had to extend the entire class to do this...time to move along.

Please let me know if you have any questions.

If you want to go forward, let me know and I’ll push this over to the bug reports.

Thanks for listening.

Randy

Redirect

2008-09-01T18:45:17-06:00

Hello,


redirect();

should include a new parameter (4th) which can redirect to external websites.

if ( ! function_exists('redirect'))
{
    function redirect($uri = '', $method = 'location', $http_response_code = 302, $external = false)
    {
        if($external == true)
        {
            header("Refresh:0;url=".$uri);
        }

        switch($method)
        {
            case 'refresh'    : header("Refresh:0;url=".site_url($uri));
                break;
            default            : header("Location: ".site_url($uri), TRUE, $http_response_code);
                break;
        }
        exit;
    }
}

Extending helpers with multiple files

2008-09-01T09:19:25-06:00

Instead of searching for only one file to extend the helper

$ext_helper = APPPATH.'helpers/'.config_item('subclass_prefix').$helper.EXT;

// Is this a helper extension request?
if (file_exists($ext_helper))
{
   $base_helper = BASEPATH.'helpers/'.$helper.EXT;

   if ( ! file_exists($base_helper))
   {
        show_error('Unable to load the requested file: helpers/'.$helper.EXT);
   }

   include_once($ext_helper);
   include_once($base_helper);
}

Search for multiple files

$ext_helpers = glob(APPPATH.'helpers/'.substr(config_item('subclass_prefix'),0,-1).'*'.$helper.EXT);

// Is this a helper extension request?
if (count($ext_helpers) > 0)
{
   $base_helper = BASEPATH.'helpers/'.$helper.EXT;

   if ( ! file_exists($base_helper))
   {
        show_error('Unable to load the requested file: helpers/'.$helper.EXT);
   }
   
   foreach($ext_helpers as $ext_helper)
   {
       include_once($ext_helper);
   }
   include_once($base_helper);
}

The great thing about this change is that you don’t have to merge extended helper files manually anymore. Another advantage is people who use extended helpers can add their own prefix to distinguish their extension from others: MY_auth1_url_helper.php, MY_auth2_url_helper.php, ...

The dark side of the modification is the possibility functions have the same name in different files and thus creating an error when they are loaded.

Comments are welcome.