search

SemicolonAcademy / ci-cms

Automatically exported from code.google.com/p/ci-cms
Other
0 stars 0 forks source link

Code review request #23

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Purpose of code changes on this branch: Adding ajax photo gallery to ci-cms.

When reviewing my code changes, please focus on: installation and application.

After the review, Please merge this branch into:
/trunk/application/modules/gallery

Original issue reported on code.google.com by rmorga...@gmail.com on 11 Jun 2009 at 3:23

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by heriniai...@gmail.com on 11 Jun 2009 at 7:16

GoogleCodeExporter commented 9 years ago 
Hello,

It seems that the gallery module you submitted did not work at first install. 
It is
not complete yet is it?

Original comment by heriniai...@gmail.com on 11 Jun 2009 at 8:14

GoogleCodeExporter commented 9 years ago 
I am using it on a current site. What errors did you get/ Send them to me and 
I'll
track it down.

Original comment by rmorga...@gmail.com on 18 Jun 2009 at 6:22

GoogleCodeExporter commented 9 years ago 
After installation I click on the menu Gallery in admin i.e.
http://ci-cms/admin/gallery/listall

Ity gives me

A PHP Error was encountered

Severity: Warning

Message: Invalid argument supplied for foreach()

Filename: admin/list_all_view.php

Line Number: 33

Original comment by heriniai...@gmail.com on 23 Jun 2009 at 7:36

GoogleCodeExporter commented 9 years ago 
hi,

put it like this, you can shorted out your problem

if($images){

foreach()
{

}
}

thanks 

regards
A.R.Ramamoorthy
ramamoorthymsc@gmail.com

Original comment by johnmatt...@gmail.com on 18 Jul 2009 at 10:30