laravel - if else loop not working proper

Question

I'm trying to write an if-else loop: if there is an embed saved in the database it should display a calendar, if not it should display a button saying "add new calendar/add new embed", but it's not working. I can't figure out why. When there is an embed saved in the database it displays the embed but, if there isn't an embed it doesn't display the button to add a new calendar/embed:

Blade view:

@if (empty($calendars))

@else
    <h4>No calendar settings</h4><br>
    <a href="{{ route('calendarsettings.create')}}">
        <button class="btn btn-success btn-cons m-b-10" type="button">
            <i class="fa fa-cloud-upload"></i>
            <span class="bold">Add embed</span>
        </button>
    </a>
@endif
@foreach($calendars as $calendar)
    {{ $calendar->embed }}
@endforeach

Controller:

public function kalendarz() {
    $calendars = Calendar::with('users')->where('user_id', Auth::user()->id)->get();
    return view('kalendarz',['calendars'=>$calendars]);
}

Blade code block tells that "if there is no calendars do nothing" — Dmitriy Troian
– Dmitriy Troian, Commented Apr 9, 2017 at 7:43

Jean-Philippe Murray · Accepted Answer · 2017-04-09 11:55:39Z

1

In Laravel 5.4, I would probably use the @unless blade directive, that will show the block, unless a condition is met. What the following code would acheive is "show the entries if we have some, and show the button unless we have some entries".

@if($calendar)
    // show entries
@endif

@unless(empty($calendars))
    // show a button to add a new one
@endunless

It kind of is the same "logic" as with an else, but somehow I think it's more clear and easy to read than an @else statement, and make it separate from the if.

answered Apr 9, 2017 at 11:55

Jean-Philippe Murray

1,2482 gold badges12 silver badges35 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

imrealashu · Accepted Answer · 2017-04-09 08:27:45Z

1

The correct syntax is:

@if(empty($calendars))
    <h4>No calendar settings</h4></br>
    <a href="{{ route('calendarsettings.create')}}">
        <button class="btn btn-success btn-cons m-b-10" type="button"><i class="fa fa-cloud-upload"></i> <span class="bold">Add embed</span></button>
    </a>
@else
    @foreach($calendars as $calendar)
        {{ $calendar->embed }}
    @endforeach
@endif

edited Apr 9, 2017 at 8:27

imrealashu

5,0994 gold badges18 silver badges28 bronze badges

answered Apr 9, 2017 at 7:43

Alexey Mezenin

164k27 gold badges309 silver badges287 bronze badges

6 Comments

tomczas Over a year ago

still nothing if there's no calendar. In database embed is text field

Alexey Mezenin Over a year ago

@tomczas please show result of {{ dd($calendars) }}

tomczas Over a year ago

it displays nothing

tomczas Over a year ago

that's how database to store embed look like

Schema::create('calendar', function (Blueprint $table) {           $table->increments('id');           $table->integer('user_id')->unsigned();           $table->string('embed');           $table->timestamps();       });        Schema::table('calendar', function (Blueprint $table) {     $table->foreign('user_id')->references('id')->on('users');     });

tomczas Over a year ago

any ideas? @Alexey Mezenin

|

Advaith · Accepted Answer · 2017-04-09 08:34:51Z

0

empty() functions empty the variable $calendars, so try using count()

@if(count($calendars) > 0)
    <h4>No calendar settings</h4></br>
    <a href="{{ route('calendarsettings.create')}}">
        <button class="btn btn-success btn-cons m-b-10" type="button"><i class="fa fa-cloud-upload"></i> <span class="bold">Add embed</span></button>
    </a>
@else
    @foreach($calendars as $calendar)
        {{ $calendar->embed }}
    @endforeach
@endif

answered Apr 9, 2017 at 8:34

Advaith

2,5903 gold badges23 silver badges38 bronze badges

Comments

Marwelln · Accepted Answer · 2017-04-09 08:42:27Z

0

When using Eloquent to get results from a database you get an instance of Illuminate\Database\Eloquent\Collection. To check if that collection is empty you use $collection->isEmpty() or $collection->isNotEmpty() (L5.4).

@if ($calendars->isNotEmpty())
    // Have entries.
@else
<h4>No calendar settings</h4></br>
<a href="{{ route('calendarsettings.create')}}"><button class="btn btn-success btn-cons m-b-10" type="button"><i class="fa fa-cloud-upload"></i> <span class="bold">Add embed</span>
   </button></a>
@endif

@foreach($calendars as $calendar)
    {{$calendar->embed}}
@endforeach

answered Apr 9, 2017 at 8:42

Marwelln

29.5k21 gold badges97 silver badges120 bronze badges

Collectives™ on Stack Overflow

laravel - if else loop not working proper

4 Answers 4

Comments

6 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

6 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related