Rymercyble

c_id is null because you are missing value of your country select

<option value ="">

also naming is wrong your option menu got name country and state what is key to access data passed from this input but you are trying to access it via s_name

hello, i have got tables articles, files and uploads

my current query

ArticleNew::where('article_id', 2931)
            ->with([
                'files' => function($query){
                    $query->select('url', 'filename', 'mimetype', 'db_files_new.file_id')
                        ->orderBy('file_id');
                },
                'uploads' => function($query){
                    $query->where('thumbnail_id', '!=', 0)
                        ->select('db_uploads_new.file_id', 'thumbnail_id');
                },
                'category',
                'relations',
            ])->select('article_id', 'lang_id', 'url', 'author_id', 'created_on', 'title', 'description', 'content')
            ->get();

it returns article itself and related files (images/videos) but if it is video then it got thumbnail paired via uploads table but this returns only thumbnail_id from uploads table and i also need to get data from files table for that thumbnail_id

sorry my inattention, i'm migrating old database structure to new but old system wasn't deleting relations from pivot table and then it looks like there should be 5 files but there is actually just 1

@ftiersch that makes no difference even if i delete whole method from storedfiles it still returns just 1 i tried every order of keys in storedfiles model and still getting just 1

hello, my database tables are db_articles, db_articles_files, db_files and keys are article_id and file_id

models

class StoredFiles extends Model
{
    protected $table = 'db_files';

    public function article(){
        return $this->belongsToMany('App\ArticleNew', 'db_articles_files', 'article_id', 'file_id', 'article_id', 'file_id');
    }
}

class ArticleNew extends Model
{
    protected $table = 'db_articles_new';

    public function storedFiles(){
        return $this->belongsToMany('App\StoredFiles', 'db_articles_files', 'article_id', 'file_id', 'article_id', 'file_id');
    }
}

but

ArticleNew::where('article_id', 125)->with('storedFiles')->get();

returns just 1 related file but there should be 5 files for this article

again my mistake - i can use select but i also have to select id otherwise knownlanguage method cant find relations

is there other way except pluck ?

its kind of working if i do

User::with('knownLanguage')->get();

its working but when i add select method

User::with('knownLanguage')
                ->select('hidden_id', 'nick_name', 'first_name', 'last_name', 'date_of_birth', 'about_me', 'profile_photo', 'country', 'gender')
                ->get();

its not working

I found where was the problem - pivot table was empty due to error during saving of data which i didnt notice

not every user got language but it returns empty array even for users who got it

Hello, I would like to get every user with every language they know I have got tables users, languages, and known_language_user

in user model defined relation

public function knownLanguage()
    {
        return $this->belongsToMany('App\Language', 'known_language_user', 'user_id', 'language_id');
    }

i tried

User::with('knownLanguage')->get();

but it gives me empty array

"known_language": []

where i don't even know why the name of key is split by "_"

than-q for answers

Hello, I would be very happy if someone turns me to the right way. At the moment I'm using controllers probably in the wrong way (I'm quite not sure) When I need for example manage articles I'm using 1 controller for /articles (get,post) - list of all or filtered articles /article (get,post) - showing 1 whole specific article and often even more than 4 methods. Is that ok or should I divide it ? Also, I'm probably putting too much logic to my methods for example

public function people()
    {
        $search = Request::has('search') ? Request::get('search') : 0;
        if ($search) {
            if (Request::get('min_age') == ''){
                $min_age = '10';
            } else {
                $min_age = Request::get('min_age');
            }
            if (Request::get('max_age') == ''){
                $max_age = '120';
            } else {
                $max_age = Request::get('max_age');
            }

            $carbon = Carbon::today();
            $carbon_max = Carbon::today();
            $max_age += 1;
            $year_min = $carbon->year - $min_age;
            $year_max = $carbon_max->year - $max_age;
            $carbon_min = $carbon->setDate($year_min, $carbon->month, $carbon->day)->toDateString();
            $carbon_max = $carbon_max->setDate($year_max, $carbon->month, $carbon->day)->toDateString();

            $users = User::when(!empty(Request::get('country')),function ($query){
                $query->where('country', Country::where('name', Request::get('country'))->select('id')->first()->id);
            })->when(!empty(Request::get('physical_gender')),function ($query){
                $query->where('physical_gender', Physical_gender::where('name', Request::get('physical_gender'))->select('id')->first()->id);
            })->when(!empty(Request::get('language')),function ($query){
                $query->whereHas('languages', function($query){
                    $query->where('name', Request::get('language'));
                });
            })->when(!empty(Request::get('language_want')),function ($query){
                $query->whereHas('unknownLanguages', function($query){
                    $query->where('name', Request::get('language_want'));
                });
            })->whereBetween('date_of_birth', [$carbon_max, $carbon_min])
                ->select('nick_name','first_name','last_name','date_of_birth','hidden_id')
                ->get();
        } else {
            $users = Sentinel::getUserRepository()->select('nick_name','first_name','last_name','date_of_birth','hidden_id')->get();
        }

        return view('people', compact('users'));
    }

if this is wrong where i should store logic ?

@JLRDW - but I'm not using login system

I solved it like this

public function postTrade(Request $request){
    $key = "key";
        Session::put($key, $request->all());
        return redirect('/stage2')->with(['key' => $key]);
    }

<script>
    sessionStorage.setItem("key", "{{ session('key') }}");
</script>
<form action="" method="POST">
    @csrf
    <input type="hidden" name="key" id="key">
    <script>
        document.getElementById("key").value = sessionStorage.getItem("key");
    </script>
    <input type="submit" name="pay" value="Pay">
</form>

public function postTradeFinish(Request $request){
        $key = $request->key;
        return Session::get($key);
    }

Hello, I would appreciate feedback to logic behind my shopping cart. At /stage1 user fills data like name address etc. which i store as temporary session. Then I redirect him to /stage2 with key to that session. At /stage2 is payment wall. With JS I catch the key and store it locally then retrieve it and insert it to hidden input. After clicking "pay" I retrieve session in controller using key from hidden input to pair the user's data with payment.

@JLRDW - how to manage them when there will be for example 50 sessions existing at same time ? i need unique name for each session

hello, at /stage1 is post form via which I'm getting data which I need to store in db but not until confirmating via post form at /stage2

Route::post('/stage1', '[email protected]');

public function postTrade(Request $request){
        return redirect('/stage2')->with(['data' => $request->all()]);
    }

Route::get('/stage2', '[email protected]');

public function getTradeFinish(){
        $data = Session::has('data') ? Session::get('data') : [];
        return view('tradeFinish')->with(['data' => 'data']);
    }

to this point it works good but when I submit post at /stage2 it gives me 419 Page Expired

edit - 419 Page Expired was my mistake now i'm getting empty array

Route::post('/stage2', '[email protected]');

public function postTradeFinish(){
        return Session::has('data') ? Session::get('data') : [];
    }

any advice ?

i made mistake in different place

hello i'm making middleware to control access based on users role i'm checking if currently logged user has role with role id higher than 100

if(Sentinel::check()){
            if(Sentinel::getUser()->roles()->select('role_id')->first()->role_id >= 100){
                return $next($request);
            } else {
                return redirect()->back()->with(['error' => 'Access denied']);
            }
        } else {
            return redirect ('/login')->with(['error' => 'You are not logged in.']);
        }

but

Sentinel::getUser()->roles()->select('role_id')->first()->role_id >= 100;

return boolean what results in exception The Response content must be a string or object implementing __toString(), "boolean" given.

is there a way to allow boolean in middleware or different way how to check this ?

Hello, I would like to know if doing things like this is worth it. This check if conversation already exist and if no then I save it so i can show every conversation for each user without need of sending query to table with all messages and conversation ids they belongs to

if(Conversation::where('conversation_id', $new_chat_id)->first() != null){
            return redirect("/messages/{$new_chat_id}");
        } else {
            Conversation::insert([
                'conversation_id' => $new_chat_id,
                'user1' => $id,
                'user2' => $recipient
            ]);
            return redirect("/messages/{$new_chat_id}");
        }

hello i'm making something like social site,

users profiles are /users/id

i would like to hide user's ids behind something like hash but in hexadecimal form with every time same length

ideal would be if it cant be retrieved back to original number

yes i could just attach random number to each user but there would be every time check if that number isn't already attached to another user i tried $id_transform = hexdec( substr ( md5 ( md5 ( $id_transform )) , 10 , -10 )); but that doesn't create unique and same length strings

@arukomp how to change this

User::whereHas('languages', function($query){
                $query->where('name', Request::get('language'));
            })

to when(!empty... ? this relation is m:n with language_user pivot

@vilfago I'm not sure what you mean ?

I want to make user filter search. For all users I can use

User::all();

And if i want to filter them by country I need

User::where('country', Country::where('name', $country)->select('id')->first()->id)

but this don't handle situation when user won't specify country so I need to change it to something like this

if (Request::get('country') == ''){
                $country_e = '!=';
                $country = 'default';
            } else {
                $country_e = '=';
            }

User::where('country', $country_e, Country::where('name', $country)->select('id')->first()->id)

But this way I'm making pointless query so I can devide each combination which user can search in separated if and this way i would have to use like 100 ifs to handle every combination

So how to make query where "where" would count only if data which each "where" search was received

I have table users and table language and I want to make pivot for languages users know and don't know but want to learn them so I need pivots language_user and unknown_language_user but this don't work because query for unknown_language_user want to join unknown_language table which don't exist. Is there a way how to do it without making separated table with same data ?

@topvillas did you mean something like this ?

private function getProfile($id){
        return User::where('hidden_id', $id)->first();
    }

    private function getInfo(){
        $profile = $this->getProfile();
        $id = $profile->id;
        return $info = [
            'physical_gender' => $profile->show_physical_gender == 1 ? User::find($id)->physicalGender()->select('name')->first() : null,
            'mental_gender' => $profile->show_mental_gender == 1 ? User::find($id)->mentalGender()->first() : null,
            'sexuality' => $profile->show_sexuality == 1 ? User::find($id)->sexuality()->first() : null,
            'smoking' => $profile->show_smoke == 1 ? User::find($id)->smoking()->first() : null,
            'alcohol' => $profile->show_alcohol == 1 ? User::find($id)->alcohol()->first() : null,
            'language_i_know' => $profile->show_language_i_know == 1 ? User::find($id)->languages()->select('name','level')->get() : null,
            'language_i_want_to_learn' => $profile->show_language_i_want_to_learn == 1 ? User::find($id)->unknownLanguages()->get() : null,
            'relationship_status' => $profile->show_relationship_status == 1 ? User::find($id)->relationship()->first() : null,
            'children_status' => $profile->show_children_status == 1 ? User::find($id)->children()->first() : null,
            'search_status' => $profile->show_search_status == 1 ? User::find($id)->search()->first() : null,
            'religion' => $profile->show_religion == 1 ? User::find($id)->religion()->first() : null,
            'country' => $profile->show_country == 1 ? User::find($id)->countries()->first() : null,
            'city' => $profile->show_city == 1 ? User::find($id)->city()->first() : null,
        ];
    }

    private function showProfile($id){
        $profile = $this->getProfile();
        $info = $this->getInfo();
        if (!$profile){
            return (['error', 'No such profile exist.']);
        }
        if(Sentinel::getUser()->id != $id){
            return view('profile', ['profile' => $profile, 'info' => $info, 'change' => '']);
        } elseif(Sentinel::getUser()->id == $id){
            return view('profile', ['profile' => $profile, 'info' => $info, 'change' => 'change']);
        }
    }

    public function changeProfile(Request $request){
        $profile = $this->getProfile();
        $info = $this->getInfo();
        if ($request->input('save') == 'Save'){
            $input = $request->all();
            if($input['nick_name'] != $profile->nick_name){
                $user = User::where('nick_name', $profile->nick_name)->first();
                $user->nick_name = $input['nick_name'];
                $user->save();
            }

            return redirect()->back()->with(['announcement', 'Your profile was updated.']);
        }
    }

this is my controller

class profileController extends Controller
{
    public function showProfile(Request $request, $id){
        $profile = User::where('hidden_id', $id)->first();
        if (!$profile){
            return (['error', 'No such profile exist.']);
        }
        $id = $profile->id;
        $info = [
            'physical_gender' => $profile->show_physical_gender == 1 ? User::find($id)->physicalGender()->select('name')->first() : null,
            'mental_gender' => $profile->show_mental_gender == 1 ? User::find($id)->mentalGender()->first() : null,
            'sexuality' => $profile->show_sexuality == 1 ? User::find($id)->sexuality()->first() : null,
            'smoking' => $profile->show_smoke == 1 ? User::find($id)->smoking()->first() : null,
            'alcohol' => $profile->show_alcohol == 1 ? User::find($id)->alcohol()->first() : null,
            'language_i_know' => $profile->show_language_i_know == 1 ? User::find($id)->languages()->select('name','level')->get() : null,
            'language_i_want_to_learn' => $profile->show_language_i_want_to_learn == 1 ? User::find($id)->unknownLanguages()->get() : null,
            'relationship_status' => $profile->show_relationship_status == 1 ? User::find($id)->relationship()->first() : null,
            'children_status' => $profile->show_children_status == 1 ? User::find($id)->children()->first() : null,
            'search_status' => $profile->show_search_status == 1 ? User::find($id)->search()->first() : null,
            'religion' => $profile->show_religion == 1 ? User::find($id)->religion()->first() : null,
            'country' => $profile->show_country == 1 ? User::find($id)->countries()->first() : null,
            'city' => $profile->show_city == 1 ? User::find($id)->city()->first() : null,
        ];
        if(Sentinel::getUser()->id != $id){
            return view('profile', ['profile' => $profile, 'info' => $info, 'change' => '']);
        } elseif(Sentinel::getUser()->id == $id){
            return view('profile', ['profile' => $profile, 'info' => $info, 'change' => 'change']);
        }
    }

    public function changeProfile(Request $request){
        if ($request->input('save') == 'Save'){
            $input = $request->all();
            if($input['nick_name'] != $profile->nick_name){
                $user = User::where('nick_name', $profile->nick_name)->first();
                $user->nick_name = $input['nick_name'];
                $user->save();
            }

            return redirect()->back()->with(['announcement', 'Your profile was updated.']);
        }
    }
}

and i need to share $profile and $info to changeProfile method so i don't need to repeat query

I have stored unique info about users in table users and other info (which many users have same like gender) in separated tables. In users table i have stored also info if user want to show his info on his profile thats why i have so many ifs in controller i can't do something like what @breki said because i have to check firstly if user want to show where he live and if not i don't need to start query to get his adress

my profile controller

public function showProfile(Request $request, $id){
        $profile = User::where('hidden_id', $id)->first();
        $id = $profile->id;

        if($profile->show_physical_gender == 1){
            $physical_gender = User::find($id)->physicalGender()->first();
        } else {
            $physical_gender = null;
        }
        if($profile->show_mental_gender == 1){
            $mental_gender = User::find($id)->mentalGender()->first();
        } else {
            $mental_gender = null;
        }
        if($profile->show_sexuality == 1){
            $sexuality = User::find($id)->sexuality()->first();
        } else {
            $sexuality = null;
        }
        if($profile->show_smoke == 1){
            $smoking = User::find($id)->smoking()->first();
        } else {
            $smoking = null;
        }
        if($profile->show_alcohol == 1){
            $alcohol = User::find($id)->alcohol()->first();
        } else {
            $alcohol = null;
        }
        if($profile->show_language_i_know == 1){
            $language_i_know = User::find($id)->languages()->select('name','level')->get();
        } else {
            $language_i_know = null;
        }
        if($profile->show_language_i_want_to_learn == 1){
            $language_i_want_to_learn = User::find($id)->unknownLanguages()->get();
        } else {
            $language_i_want_to_learn = null;
        }
        if($profile->show_relationship_status == 1){
            $relationship_status = User::find($id)->relationship()->first();
        } else {
            $relationship_status = null;
        }
        if($profile->show_children_status == 1){
            $children_status = User::find($id)->children()->first();
        } else {
            $children_status = null;
        }
        if($profile->show_search_status == 1){
            $search_status = User::find($id)->search()->first();
        } else {
            $search_status = null;
        }
        if($profile->show_religion == 1){
            $religion = User::find($id)->religion()->first();
        } else {
            $religion = null;
        }
        if($profile->show_country == 1){
            $country = User::find($id)->countries()->first();
        } else {
            $country = null;
        }
        if($profile->show_city == 1){
            $city = User::find($id)->city()->first();
        } else {
            $city = null;
        }
        $info = [
            'physical_gender' => $physical_gender,
            'mental_gender' => $mental_gender,
            'sexuality' => $sexuality,
            'smoking' => $smoking,
            'alcohol' => $alcohol,
            'language_i_know' => $language_i_know,
            'language_i_want_to_learn' => $language_i_want_to_learn,
            'relationship_status' => $relationship_status,
            'children_status' => $children_status,
            'search_status' => $search_status,
            'religion' => $religion,
            'country' => $country,
            'city' => $city,
        ];
        if(Sentinel::getUser()->id != $id){
            return view('profile', ['profile' => $profile, 'info' => $info, 'change' => '']);
        } elseif(Sentinel::getUser()->id == $id){
            if (Request::get('change') != 'Change'){
                return view('profile', ['profile' => $profile, 'info' => $info, 'change' => 'able']);
            } elseif (Request::get('change') == 'Change'){
                return view('profile', ['profile' => $profile, 'info' => $info, 'change' => 'change']);
            }
        }
    }

my view

if($change == 'able'){
        ?>
        <form action="" method="GET">
            <input type="submit" name="change"  value="Change">
        </form>
        <?php
    }
    if($change == 'change'){
        ?>
        <form action="" method="GET">
            <input type="submit" name="save"  value="Save changes">
        </form>
        <?php
    } else {
        if($profile->name_decision == 0){
            echo $profile->nick_name;
        } elseif($profile->name_decision == 1){
            echo $profile->first_name;
        } elseif($profile->name_decision == 2){
            echo $profile->first_name, $profile->last_name;
        }
        if($info['physical_gender'] != null){
            echo '</br>'.ucfirst($info['physical_gender']->name);
        }
        if($info['mental_gender'] != null){
            echo '</br>'.ucfirst($info['mental_gender']->name);
        }
        if($info['sexuality'] != null){
            echo '</br>'.ucfirst($info['sexuality']->name);
        }
        if($info['smoking'] != null){
            echo '</br>'.ucfirst($info['smoking']->name);
        }
        if($info['alcohol'] != null){
            echo '</br>'.ucfirst($info['alcohol']->name);
        }
        if($info['language_i_know'] != ""){
            foreach($info['language_i_know'] as $lang){
                echo '</br>'.ucfirst($lang->name).' '.$lang->level;
            }
        }
        if($info['language_i_want_to_learn'] != ""){
            foreach($info['language_i_want_to_learn'] as $lang2){
                echo '</br>'.ucfirst($lang2->name);
            }
        }
        if($info['relationship_status'] != null){
            echo '</br>'.ucfirst($info['relationship_status']->name);
        }
        if($info['search_status'] != null){
            echo '</br>'.ucfirst($info['search_status']->name);
        }
        if($info['religion'] != null){
            echo '</br>'.ucfirst($info['religion']->name);
        }
        if($info['country'] != null){
            echo '</br>'.ucfirst($info['country']->name);
            if($info['city'] != null){
                echo ' - '.ucfirst($info['city']->name);
            }
        }
    }

@GertjanRoke i have one additional question - for languages i have 2 tables one for languages which user know and one for languages which user want to learn. How it would be in that case ?

@GertjanRoke thanks

@GertjanRoke it returns me strpos() expects parameter 1 to be string, array given

yes for example yes

I have users table with data like nickname, date of birth etc. Then I have tables like countries, country_user, languages, language_user. So I can get request like "find all users by age and country" or any combinations like this.

I have no idea how to make query for this. I'm not sure if it is good type of question for this forum but I've been searching at google for hours and nothing found.

i want to do things like if you are admin and you look at user profile for you there will be option to ban that user or in chat room admin will have option to delete message of any user

i can make middleware like this

if($role == 'member'){
    return redirect('/people/profile/{id}');
 } else {
    return redirect('/people/profile/{id}')->with(['access' => 'admin']);
}

but this is solution only for one view so i need unique middleware for every view otherwise i can make role check in each view both options are bad

so exist any option how to make universal middleware for this ?

something what do same thing as this

if($role == 'member'){
    return $next($request);
 } else {
    return $next($request);->with(['access' => 'admin']);
}

i'm trying to make middleware to controll content based on roles and

this

return $next($request)->with(['access' => 'owner']);

throw BadMethodCallException Method with does not exist.

is there any way how to do something with same result ?

@martinbean by this i wanted to avoid this

If there would be 1000 users and everyone will write to everyone e.g. 100 messages then i would like to have 499 500 tables instead of table with 49 950 000 messages which i will need to "read" everytime when someone would want to read his messages

can be table created automatically ? i want to create table for every conversation between 2 users so i need something like this

$id1= 1;
$id2= 2;

$table_name = $id1."_".$id2;

php artisan:make migration $table_name

Schema::create($table_name, function (Blueprint $table) {
            $table->integer('id')->unsigned();
            $table->text('message');
        $table->timestamps();

            $table->engine = 'InnoDB';
        });

php artisan migrate

and delete that migration

but all automatically

i use

DB::table('users')->whereBetween('date_of_birth', [$carbon_max, $carbon_min])->get();

if $carbon_max and $carbon_min was for example 10 and 1 this would return 9,8,7,6,5,4,3,2 but i also need 10 and 1

but this

$users = DB::table('users')
                ->where('date_of_birth', '>=', $carbon_min)
                ->where('date_of_birth', '<=', $carbon_max)
                ->get();

don't work for me

solved by moving

return view('people', compact('users'));

inside if

with

$users = [];

before if it return empty array

my controller

public function people()
    {
        $min_age = Request::has('min_age') ? Request::get('min_age') : null;
        $max_age = Request::has('max_age') ? Request::get('max_age') : null;

        if (isset($min_age) && isset($max_age))
        {
            $carbon = Carbon::today();
            $carbon_max = Carbon::today();
            $year_min = $carbon->year - $min_age;
            $year_max = $carbon_max->year - $max_age;
            $month = $carbon->month;
            $day = $carbon->day;
            $carbon_min = $carbon->setDate($year_min, $month, $day)->toDateString();
            $carbon_max = $carbon_max->setDate($year_max, $month, $day)->toDateString();
            $users = DB::table('users')->whereBetween('date_of_birth', [$carbon_min, $carbon_max])->get();
        }
        return view('people', compact('users'));
    }

my view

<?php foreach ($users as $user) { ?>
                <div class="people_container">
                    <a href="/people/<?php echo $user->nick_name; ?> "><?php echo $user->nick_name; ?></a><br>
                    <?php echo $user->first_name; ?>
                    <?php echo $user->last_name; ?>
                </div>
            <?php } ?>

it gives

ErrorException Undefined variable: users in foreach in view