[L4] E-Commerce question, about products...

If I select "black" I know from the db that the size "14-16" is the only one that has stock, but yet shows all three sizes.

DB:

I had a good read through the docs and f**k me I found

->having('stock', '>', 0)

So added that to the "selectedColour" method and now works kinda, but if i have colours black, white & green with only black size 14-16 with some stock the other colours still show, how can make this only show colours with stock.

So in this example the colour select/drop down should only show Black, has black is the only colour that as an item with stock more than zero

Currently showing like this which isn't right.

Anyone know based on everything I have posted and talked about how I can do this?

Still showing incorrectly need a second pair of eyes on this if someone can help

Still looking for advice on this if anyone can help.

Anyone?

In your product details don't you need populate the options with availableColours rather than colour?

Not sure what you mean, i have:

 @if($item->availableColours() && $item->availableStock())
                                Colour
                                <select name="colour" class="colors" data-index="{{ $item->id }}">
                                    <option value="" selected="selected">Please Select</option>
                                    @foreach($item->options as $option)
                                        <option value="{{ $option->product_id }}-{{ $option->colour }}">{{ $option->colour }}</option>
                                    @endforeach
                                </select>
                            @endif

So you're only showing products that have stock for the available colour but it's displaying all colours here:

<option value="{{ $option->product_id }}-{{ $option->colour }}">{{ $option->colour }}</option>

$option->colour needs to only list available colours rather than all colours, is that right?

Something along these lines:

<option value="{{ $option->product_id }}-{{ $option->availableColours }}">{{ $option-> availableColours }}</option>

yes thats correct

Something like:

<option value="{{ $option->product_id }}-{{ $option->availableColours->colour }}">{{ $option-> availableColours->colour }}</option>

That produces Trying to get property of non-object

still no joy :(

arrrgggghhhh :-) this is driving me mad

You're foreach looping through @foreach($item->options as $option) so options must be returning colours that aren't available. I wonder if something like @foreach($item->options->availableColours as $option)

Or would you be better with an additional method that only returns availableOptions

Undefined property: Illuminate\Database\Eloquent\Collection::$availableColours when doing @foreach($item->options->availableColours as $option) so i guess an additional method would be better, just not sure how :( I know its driving me crazy too

I'm still thinking we might be able to use availableColours somehow.

<option value="{{ $option->product_id }}-{{ $option->availableColours }}">{{ $option-> availableColours }}</option>

or

<option value="{{ $option->product_id }}-{{ $option->availableColours() }}">{{ $option-> availableColours() }}</option>

The first option doesn't display anything and the second option display a relationship error

That's why i don't all this Mambo Jambo (on the fly list) although, i have to admit, it's nice.

I'm doing it the good ol' way showing all products available either in a dropdown list or listed with radio buttons. Three options: Item in stock, out of stock, but will get in soon, no more available.

Although i usually don't like to mix more options, ie there's a shirt in red, availble in sizes S-XXl, another shirt sizes S-XXL etc. don't like: this shirt is availble in red, green, yellow and in sizes S-XXL, etc.

I just spotted a couple of erroneous spaces in my $option-> availableColours(), count that will make a big difference though.

i corrected the spaces, still same errors

Back to the new method plan, something like:

public function optionsWithAvailableColours()
{
    return $this->hasMany('Options','product_id', 'id')->select('colour')->distinct()->where('stock', '>', 0)->lists('colour');
}

arrrrggghhhh

Call to undefined method Illuminate\Database\Query\Builder::optionsWithAvailableColours() 

lol, where is that method? Can you share your product model?

class Products extends \Eloquent {

    protected $guarded = ['id'];

    public function categories()
    {
        return $this->belongsTo('Categories');
    }

    public function options()
    {
        return $this->hasMany('Options','product_id', 'id');
    }

    public function availableColours()
    {
        return $this->options()->select('colour')->distinct()->where('stock', '!=', 0)->lists('colour');
    }

    public function availableSizes()
    {
        return $this->options()->select('size')->distinct()->where('stock', '!=', 0)->lists('size');
    }

    public function availableStock()
    {
        return $this->options()->select('stock')->distinct()->where('stock', '!=', 0)->lists('stock');
    }

    public function optionsWithAvailableColours()
    {
        return $this->hasMany('Options','product_id', 'id')->select('colour')->distinct()->where('stock', '>', 0)->lists('colour');
    }

}

what about the html/blade

something like:

<select name="colour" class="colors" data-index="{{ $item->id }}">
        <option value="" selected="selected">Please Select</option>
        @foreach($item->optionsWithAvailableColours as $option)
            <option value="{{ $option->product_id }}-{{ $option->colour }}">{{ $option->colour }}</option>
        @endforeach
   </select>

Relationship method must return an object of type Illuminate\Database\Eloquent\Relations\Relation

ok, scratch those.

Can you try and die & dump options and availableColours so we can see what's being passed to the page.

dd outputs

array (size=1) 0 => string 'Black' (length=5)

but it needs to be $item->options as I'm using the options relationship