Laravel 4 - Eloquent one to many banner clicks - laravel

Question

I can't see where I'm going wrong with my code. I believe I have my relationships/models set up correctly but my bannerclicks table will not insert a row, and worse it seems to break the banner redirect.
MIGRATION
const TBL_BANNER_CLICKS = 'banner_clicks';
public function up()
{
// Create the clicks table
Schema::create(self::TBL_BANNER_CLICKS, function($table){
$table->engine = 'InnoDB';
$table->increments('id');
$table->integer('banner_id')->unsigned();
$table->string('ip_address', 45);
$table->integer('clicks')->unsigned()->default(0);
$table->integer('created_at');
$table->foreign('banner_id')
->references('id')->on('banners')
->onUpdate('cascade')
->onDelete('cascade')
;
});
}
MODELS
Banner.php
public function bannerclicks()
{
return $this->hasMany('BannerClick');
}
BannerClick.php
protected $table = 'banner_clicks';
public function banner()
{
return $this->belongsTo('Banner');
}
BannerController.php
public function redirect($id)
{
try {
// Original (want to keep)
$banner = Banner::where('id', $id)
->where('active', 1)
->firstOrFail();
$banner->clicks++;
$banner->save();
// new banner_clicks table
$bannerclick = new BannerClick($id);
$bannerclick->banner_id = $id;
$bannerclick->clicks = 1;
$bannerclick->bannerclicks()->save($bannerclick);
return Redirect::to($banner->url);
} catch (Exception $ex) {
return Redirect::to('/');
}
}
HTML
<a href="{{URL::action('BannerController#redirect', $banner->id)}}" target="_blank" rel="nofollow">
<img src="{{$banner->image}}" class="img-responsive" alt="{{$banner->title}}"></a>

Answer 1

I think you should be saving your relation with something like this:
public function redirect($id)
{
try {
// Original (want to keep)
$banner = Banner::where('id', $id)
->where('active', 1)
->firstOrFail();
$banner->clicks++;
$banner->save();
// new banner_clicks table
$bannerclick = new BannerClick(array('clicks' => 1));
$banner->bannerclicks()->save($bannerclick);
return Redirect::to($banner->url);
} catch (Exception $ex) {
return Redirect::to('/');
}
}
See here: http://laravel.com/docs/eloquent#inserting-related-models
Or just do this:
$banner = Banner::where('id', $id)
->where('active', 1)
->firstOrFail();
$banner->clicks++;
$banner->save();
// new banner_clicks table
$bannerclick = new BannerClick;
$bannerclick->clicks = 1;
$bannerclick->banner_id= $banner->id;
$bannerclick->save();