Modular Extensions - HMVC version 5.2

[eluser]phpwebdev[/eluser]
[quote author="wiredesignz" date="1223422113"]Hi phpwebdev, I looked at the example application you provided and the first thing I noticed is that the system/application/controllers directory is missing.

Please re-create the controllers directory (it is absolutely required by both CI and ME) and try again. Thanks [/quote]
Jast create empty folder ?

[eluser]tdktank59[/eluser]
Im on can I get added to the list?

BTW Why have you not released this to the "public" yet. Just curious???

[eluser]wiredesignz[/eluser]
Modular Extensions 5.1 is not publicly available at this time. Probably wait until the EE 2.0 and CI release and see what needs to be added, dropped or changed.

[eluser]Sam Dark[/eluser]
Found a bug in ME 5.1.33 Loader.

To reproduce:
— Autoload library via config. ()
— Load it again manually in module controller.
— var_dump it. It will contain string 'libname' and not the library itself.

[eluser]wiredesignz[/eluser]
ME5134 bugfix sent out. Thanks Sam.

[eluser]Sam Dark[/eluser]
Found a problem with Loader case sensitivity. Occured when I've used pagination in a module. Pagination tries to load config named 'Pagination.php' but in CI user guide it's 'pagination.php' and it's correct since CI native loader is case insensitive. So my suggested way to fix it: make loader case insensitive and therefore compatible with CI.

[eluser]wiredesignz[/eluser]
That's great Sam, could you please share your method of making Loader case insensitive.

[eluser]Sam Dark[/eluser]
I think it's good to use CI native Loader way. It is in Loader.php after:

Code:

// We'll test for both lowercase and capitalized versions of the file name

[eluser]wiredesignz[/eluser]
Thanks Sam, but you just said it must be lowercase and I can assure you that double checking a filename is a waste of time.

but in CI user guide it’s ‘pagination.php’ and it’s correct since CI native loader is case insensitive

Actually CI loader is not case insensitive, it checks both ucfirst and lowercase filenames.

EDIT:
I have just checked and ME5134 libraries loading config and it works for me.

Please give some examples of where this is failing for you.

[eluser]Sam Dark[/eluser]
Will provide test case on this a little bit later.

Noticed possible bug in Loader->config(). $use_sections is not used here:

Code:

if ($path === FALSE)
{
  //I think this one should be self::$loader->config($file, $use_sections);
  self::$loader->config($file, TRUE);
                        
  return $ci->config->item($file);
}