[eluser]clip[/eluser]
[quote author="wwwald" date="1233635656"]REMARK: I'm pretty new to CI, so if my remarks don't make sense, please be gentle :-)
Anyway: in the CIG class constructor, gallery_root is already used to construct the cache_root_dir location. However, the user hasn't had the chance to set the gallery_root at this point.
Result: the cache dir will always be created with the default "albums/" name. If the user sets gallery_root to something else after loading the library, things go wrong.
I guess gallery_root should be passed to the class constructor, much like the "tw", "th", ... variables. Right?
Just shoot me if I'm babbling... just trying to help :-)
wwwald[/quote]
Thanks for pointing this out, you are absolutely correct. My intent is to not have the cache methods be called in the construct method. With the way it is now, there could be possible errors if there are in process image uploads if someone were to visit a gallery. I think I am going to make them public functions where they can be called say from a uri segment with something like:
Then all that would be needed is to call the cache methods from your controller. Or you could just call the method from the controller construct method after loading the library. I think doing this would solve the issue you pointed out and the issue just realized from your observation. I will get the new version together ASAP.
Thanks for the catch.
If anyone else has any functionality you would like to see, or bugs to point out please post them here.
