evanrolfe / test

Yacht Fractions
0 stars 0 forks source link

Admin SAVE bug #21

Closed chris31519 closed 11 years ago

chris31519 commented 11 years ago

When editing YS, save gives and error

http://php.ais.at/evan/yacht/yachtshare/handle_post

ErrorException [ Error ]: Call to undefined method stdClass::save()

APPPATH/classes/controller/yachtshare.php @ line 356

evanrolfe commented 11 years ago

Fixed and uploaded to php.ais.at so you can double check it.

evanrolfe commented 11 years ago

you can also see that required fields are now marked with a red * in ALL forms (including the admin forms)

chris31519 commented 11 years ago

what file did you change? I just downloaded the latest to my localhost and it give the same error. I was editing inactive http://localhost/yacht/yachtshare/edit/622

evanrolfe commented 11 years ago

Redownload the code from github, make sure that you are using the latest version of the code. yachtshare/edit is working fine on my localhost and on php.ais.at.

To see if you are using the latest version check the file: fuel/app/classes/controller/yachtshare.php, around line #320 there will be a comment:

//3. Or update the already existing yachtshare

check the code immediately after the comment, it should be like this:

//3. Or update the already existing yachtshare }elseif(Input::post('update')){ $yachtshare = Model_Yachtshare::find(Input::post('yachtshare_id'));

if(!is_null($yachtshare->group_ids)) { $associated_ids = explode(',', $yachtshare->group_ids); }else{ $associated_ids = array($yachtshare->id); }

$errors = 0;

foreach($associated_ids as $id) {

Its taking me little longer to do the front end site, the javascript image gallery thing is proving to be a kind of a pain, becasue I didn't code the template itself its hard to get it to fit in the right position but i will make it happen. Also I've been testing the emails and they're all working fine so thats one less thing we need to worry about.

I'm gonna finish the front-end site later tonight/tomorrow morning because I think its better if I fix those IE bugs first, FINALLY got IE8 installed on my linux machine using WINE.

On Sat, Dec 22, 2012 at 6:43 PM, chris31519 notifications@github.comwrote:

what file did you change? I just downloaded the latest to my localhost and it give the same error. I was editing inactive http://localhost/yacht/yachtshare/edit/622

— Reply to this email directly or view it on GitHubhttps://github.com/evanrolfe/test/issues/21#issuecomment-11639980.

chris31519 commented 11 years ago

Ok cool. I pasted in that code you gave me and it worked right away. Must be that I don;t have the lastest version for some reason I'll clear the cache then download again.

Well done getting IE to work with WINE i know that's tricky. You could always do the SAVE-refresh workaround if it;s a pain.

Since you are going deeper into using the sailfractions code I ought to check with Chris what agreement he had with the developers of that site. I assume that he has full rights to it but I had better check. I did discuss with him before but I should check. Focus on getting the information there - this is a demo still - if we have static images and tell him it would be a gallery that's fine for the moment

chris31519 commented 11 years ago

Should be no problem with sailfractions looks like a custom design by a one-woman band. http://www.stotlandesigns.com/portfolio -- blog on website tips 2 years old and missing links, ha! Contacted CH to be sure.