ChangeJobPosterRemoteWorkColumnToBool::down()

Complexity

Conditions	7
Paths	2

Size

Total Lines	32
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function down()
    {
        // Since the ORM doesn't support a direct boolean to text migration we
        // have to get a little creative.

        // Add a new temp string column to the table
        Schema::table('job_posters', function (Blueprint $table) {
            $table->string('remote_work_allowed_new')->default('0');
        });

        // Loop through the existing entries and populate the new column
        $jobPosters = JobPoster::all();
        if ($jobPosters) {

$jobPosters is of type Illuminate\Database\Eloquent\Collection, thus it always evaluated to true.

            foreach ($jobPosters as $jobPoster) {
                if ($jobPoster->remote_work_allowed === '1' || $jobPoster->remote_work_allowed === true) {
                    $jobPoster->remote_work_allowed_new = '1';

The property remote_work_allowed_new does not exist on App\Models\JobPoster. Did you mean remote_work_allowed_new?

                } elseif ($jobPoster->remote_work_allowed === '0' || $jobPoster->remote_work_allowed === false) {
                    $jobPoster->remote_work_allowed_new = '0';
                }

                $jobPoster->save();
            }
        }

        // Drop the old column
        Schema::table('job_posters', function (Blueprint $table) {
            $table->dropColumn('remote_work_allowed');
        });

        // Rename the new one
        Schema::table('job_posters', function (Blueprint $table) {
            $table->renameColumn('remote_work_allowed_new', 'remote_work_allowed');
        });