Laravel groupBy on relationship without array_merge - php

I'm building a Laravel 9 project. I've created a Product model and ProductOption model. My options need to be grouped by a column called region_code (which isn't always set) so that the options for the product can easily be shown. I realise that if I run $product->options->groupBy('region_code') it will give me the desired result, but I need these grouped options to be part of the Product model, thus I'm doing an array_merge.
This approach works, but I feel that it's a bit messy having to convert it to an array and merging the results back into one.
I did try doing a groupBy('region_code') in my with clause, but this didn't give me anything.
Here's my current code and output
$selectedProduct = $request->input('product');
$selectedType = $request->input('type');
$product = Product::where('slug', $selectedProduct)
->where('is_enabled', true)
->with(['options' => function ($query) use ($selectedType) {
$query->where('is_enabled', true)
->whereNotNull('region_code')
->where('slug', 'like', "%$selectedType%");
}])->first();
if (! $product) {
return response()->json([
'message' => "No product found."
], 404);
}
$product = array_merge($product->toArray(), [
'options' => $product->options->groupBy('region_code')->toArray()
]);
return response()->json([
'product' => $product
], 200);
A cleaner approach I thought I could override the options key, and do this:
$options = $product->options->groupBy('region_code');
$product->options = $options;
return response()->json([
'product' => $product
], 200);
But this then looses the grouping entirely.
How could I clean this up?
UPDATE
Below is my table schema for each model:
Product
Schema::create('products', function (Blueprint $table) {
$table->uuid('id')->primary();
$table->string('title');
$table->string('slug')->unique();
$table->string('description')->nullable();
$table->json('extra')->nullable();
$table->boolean('is_enabled')->default(1)->index();
$table->timestamps();
$table->softDeletes();
});
ProductOption
Schema::create('product_options', function (Blueprint $table) {
$table->uuid('id')->primary();
$table->foreignUuid('product_id');
$table->string('title');
$table->string('slug')->unique();
$table->string('description')->nullable();
$table->string('region_code')->nullable()->index();
$table->json('extra')->nullable();
$table->string('stripe_id')->unique();
$table->integer('quantity')->default(1);
$table->boolean('is_enabled')->default(1)->index();
$table->timestamps();
$table->softDeletes();
});

$selectedProduct = $request->input('product');
$selectedType = $request->input('type');
$product = Product::where('slug', $selectedProduct)
->where('is_enabled', true)
->with(['options' => function ($query) use ($selectedType) {
$query->where('is_enabled', true)
->whereNotNull('region_code')
->where('slug', 'like', "%$selectedType%")
//just add here
->groupBy('region_code');
}])->first();
if (!$product) {
return response()->json([
'message' => "No product found."
], 404);
}
return response()->json([
'product' => $product
], 200);

Related

Combine 2 eloquent result and update them

I have two tables that have relation each other :
1st table products migration :
Schema::create('products', function (Blueprint $table) {
$table->id();
$table->string('name');
$table->timestamps();
});
2nd table discount migration :
Schema::create('discounts', function (Blueprint $table) {
$table->id();
$table->foreignId('product_id');
$table->double('rate', 8, 2); // rates between 0.05 ~ 0.2
$table->foreignId('period_id');
$table->timestamps();
});
Product model :
/**
* Get the product associated with discount.
*/
public function discounts()
{
return $this->hasMany(Discount::class);
}
Discount model :
/**
* Get the discount associated with product.
*/
public function product()
{
return $this->belongsTo(Product::class);
}
What i've tried so far is a bit silly :
$productWithoutDiscount = Product::doesntHave('discounts')
->where([
['section', Session::get('section-budget')],
['status', 'PREPARED']
])->get();
$productWithDiscount = Product::with('discount')->whereHas('discounts', function ($query) {
$query->where('rate' ,'>', 0.05);
})->get();
$productAll = $productWithoutDiscount->merge($productWithDiscount);
Above code works, however, i also need to update them, that wont happened as the merged eloqunet instance will be converted to collection (which doesn't have access to update() method)
How can i get both the product which don't have discount AND product which have discount with above 0.05 rate and update them ?
Update 1
So i follow #Salvon's advice to use union, so these my following code atm :
$productAll = Product::with('discount')->whereHas('discounts', function ($query) {
$query->where('rate' ,'>', 0.05);
})->union(Product::doesntHave('discounts')
->where([
['section', Session::get('section-budget')],
['status', 'PREPARED']
]));
Then another problem came out, because when i execute the update method like so :
try {
$productAll->update([
'period_id' => Session::get('period');
]);
return response()->json([
'message' => 'Data successfully updated.'
], 201);
} catch (\Throwable $th) {
return response()->json([
'message' => $th->getMessage()
], 500);
}
nothing was updated and not even any error come out,
Already check the protected $fillable on my model and the 'period_id' is there.
how can I solved this ?
Update 2
Weird after i cahge the order of query like so :
$productAll = Product::doesntHave('discounts')->where([
['section', Session::get('section-budget')],
['status', 'PREPARED']
])->union(Product::with('discount')->whereHas('discounts', function ($query) {
$query->where('rate' ,'>', 0.05);
}));
The update() method works. What is the cause ?

Laravel Getting value of another table using foreign key on Blade

I have two models Batch and Notices. The foreign key is correctly formed and data is saved in the database. Now I can't show the data on my listing blade.
Notice Migration:
Schema::create('notices', function (Blueprint $table) {
$table->id();
$table->string('noticeTitle');
$table->string('noticeDesc');
$table->string('file')->nullable();
$table->unsignedBigInteger('batch_id');
$table->foreign('batch_id')->references('id')->on('batches');
$table->timestamps();
});
Batch Migration:
Schema::create('batches', function (Blueprint $table) {
$table->id();
$table->string('name');
$table->timestamps();
});
Data dump from db is:
"id" => 1
"noticeTitle" => "Quae ea est temporib"
"noticeDesc" => "<p>sa</p>"
"file" => null
"batch_id" => 2
"created_at" => "2020-07-27 16:09:52"
"updated_at" => "2020-07-27 16:09:52"
]
Notice model
public function batches()
{
return $this->belongsTo(Batch::class);
}
Batch Model
public function notice()
{
return $this->hasMany(Notice::class);
}
Notice controller
public function index(){
$notices = Notice::all();
return view('admin.notice.list')
->with('notices', $notices);
}
public function store(Request $request)
{
$this->validate($request, [
'noticeTitle' => 'required',
'noticeDesc' => 'required',
]);
$notice = new Notice;
$notice->noticeTitle = $request->input('noticeTitle');
$notice->noticeDesc = $request->input('noticeDesc');
$notice->batch_id = $request->input('targetedGroup');
if($request->hasFile('file')) {
$noticeTitle = $request->input('noticeTitle');
$filename = $request->file->getClientOriginalName();
$request->file->storeAs('public/noticeFile/additionalFiles', $noticeTitle.'_'.$filename);
$path = $noticeTitle.'_'.$filename;
$notice->file = $path;
}
try{
$notice->save();
Session::flash('success', 'Notice Saved');
return redirect()->route('notice.index');
}
catch (\Throwable $th){
Session::flash('danger', 'Something went wrong');
return redirect()->route('notice.index');
}
}
Now I want to get batch name from batch_id
First, you might want to change the function names of your relationships. Your notice has only one batch, so it should be public function batch() and a batch has several notices, so public function notices().
You can acces your relationships like any attribute, so in your blade:
#foreach($notices as $notice)
#php $batch = $notice->batch; #endphp
{{$batch->name}}
#endforeach
Or even shorter (if you don't plan on using the related batch for anything else)
#foreach($notices as $notice)
{{$notice->batch->name}}
#endforeach
This is all explained in the Laravel documentation: https://laravel.com/docs/7.x/eloquent-relationships

Laravel - Data mismatch from eager load

I am creating factories and saving the page model to the film model so its film to page one-to-many,
i've followed the docs but when im trying to save the models to each other i am getting this error
General error: 20 datatype mismatch (SQL: insert into "pages" ("id", "page_url", "film_id", "updated_at", "created_at") values (591d61cb-3090-3945-b920-ba797245cb97, http://larson.com/, bd3bab38-f8be-4674-ae5d-15e8f6b6172a, 2019-11-15 11:23:02, 2019-11-15 11:23:02))
These are the classes i am working with
Film migration
public function up()
{
Schema::create('films', function (Blueprint $table) {
$table->uuid('id')->primary();
$table->string('name');
$table->string('description');
$table->timestamps();
});
}
Pages migration
public function up()
{
Schema::create('pages', function (Blueprint $table) {
$table->bigIncrements('id');
$table->uuid('film_id')->nullable();
$table->string('page_url')->nullable();
$table->timestamps();
});
}
PagesFactory
$factory->define(Pages::class, function (Faker $faker) {
return [
'id' => $faker->uuid,
'page_url' => $faker->url,
'film_id' => factory(\App\Models\Film::class)->create()->id
];
Pages model
public function film(): BelongsTo
{
return $this->belongsTo(Film::class);
}
FilmController
*/
public function show(string $id)
{
$film = Film::with([
'pages',
'languages',
'categories',
])->findOrFail($id);
return $film;
FilmControllerTest
public function getFilmTest()
{
$film = factory(Film::class)->create();
$language = Language::where('id', 'en')->where('name', 'English')->first();
$categories = Category::where('main-cat', 'Science')->where('sub-cat', 'Fiction')->first();
$film->pages()->save(factory(Page::class)->create());
$film->languages()->attach($language->id);
$film->categories()->attach($categories->id);
$response = $this->json('GET', '/film/' . $film->id)
->assertStatus(200);
$response
->assertJson(['id' => $guestProfile->id])
->assertJson(['name' => $film->description])
->assertJson(['languages' => $film->languages->toArray()])
->assertJson(['categories' => $film->categories->toArray()])
}
when i comment out this line from the test it works fine $film->pages()->save(factory(Page::class)->create());
im abit lost on why im having this issue trying to save the models so the pages becomes part of the response... can i get some help/example please :D
The id of your pages table is set to a bigIncrements (UNSIGNED BIGINT), but in your PagesFactory you are trying to store a uuid.
$factory->define(Pages::class, function (Faker $faker) {
return [
'id' => $faker->uuid,
'page_url' => $faker->url,
'film_id' => factory(\App\Models\Film::class)->create()->id
];
Remove 'id' => $faker->uuid, from the factory, you don't have to set an auto incrementing field.
Another option (depending on the design you have in mind) is to change the migration of the pages table and set the id column to $table->uuid('id')->primary();
try using the make() method, as in:
$film->pages()->save(factory(Page::class)->make());

SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'post_id' cannot be null

I need some help with this...I'm making a function for tags with a tutorial, but in my database ( post_tag ), my post_id isn't saved, only tag_id and id.In my posts table, after create a post I receive the id of post, but here, in my post_tag isn't.Do you know why guys...?
My controller
public function create(){
$tags = Tag::all();
return view('posts.create')->withTags($tags);
}
public function store(Request $request )
{
$data = request()->validate([
'caption' => 'required|max:255',
'image' => 'required|image',
]);
$post = new Post;
$post->tags()->sync($request->tags, false);
$imagePath = request('image')->store('uploads', 'public');
$image = Image::make(public_path("storage/{$imagePath}"))->fit(1600, 1100);
$image->save();
auth()->user()->posts()->create([
'caption' => $data['caption'],
'image' => $imagePath,
]);
return redirect('/profile/' . auth()->user()->id);
}
My create_post_tag_table
public function up()
{
Schema::create('post_tag', function (Blueprint $table) {
$table->bigIncrements('id');
$table->integer('post_id')->unsigned();
$table->foreign('post_id')->references('id')->on('posts');
$table->integer('tag_id')->unsigned();
$table->foreign('tag_id')->references('id')->on('tags');
});
}
My create_posts_table
public function up()
{
Schema::create('posts', function (Blueprint $table) {
$table->bigIncrements('id');
$table->unsignedBigInteger('user_id');
$table->string('caption');
$table->string('image');
$table->timestamps();
$table->index('user_id');
});
}
You have created a new Post object, but you have not saved it yet in this line:
$post = new Post;
Thus, in this line, immediately following:
$post->tags()->sync($request->tags, false);
There is no id in the database for this post as yet (nor will there be an id on the Post model), and thus sync will fail every time because it can't find the id.
After you new up the Post object, save() it and then you can sync.
On a different note that may help in other ways, your post model has a big int as its id:
Schema::create('posts', function (Blueprint $table) {
$table->bigIncrements('id');
}
But your post_tag table is only an integer. This mismatch may cause some issues. Suggest changing to match:
Schema::create('post_tag', function (Blueprint $table) {
$table->bigIncrements('id');
$table->integer('post_id')->unsigned(); // <--- suggest change this to bigInteger
$table->foreign('post_id')->references('id')->on('posts');
I may be wrong, but since you don't have the onUpdate or onDelete properties in your table associated with your foreign keys, it could be the problem.
The correct solution was this:
$post = auth()->user()->posts()->create([
'caption' => $data['caption'],
'image' => $imagePath,
]);
I just added this to my code, and all was ok.

How to pass to the variable the Auth::user() method and sync into pivot table?

I can get the current user that logged in. But I don't know how can I passed this into variable. I can the user id by this.
public function getDocuments()
{
//GETTING ALL THE ID OF THE USERS IN THE DATABASE EXCEPT THE ID OF CURRENT USER.
$resultRecipient = DB::table('users')->where('id', '!=', Auth::id())->get();
//GETTING ALL THE CATEGORIES.
$resultCategory = DB::table('categories')->get();
//VIEW
return view ('document.create')->with('resultRecipient', $resultRecipient)->with('resultCategory', $resultCategory);
if(\Auth::user()->id)
{
echo "You get the id";
}
else
{
echo "Failed";
}
}
Can anyone tell me how can I sync the current user id when the submit button is submitted. Is there a way how can I attach the id of the user in the sync method?
public function postDocuments(Request $request)
{
$this->validate($request,
[
'title' => 'required|alpha_dash|max:255',
'content' => 'required',
'category_id' => 'required',
'recipient_id' => 'required',
]);
$document = new Document();
//Request in the form
$document->title = $request->title;
$document->content = $request->content;
$document->category_id = $request->category_id;
$document->save();
$document->recipients()->sync($request->recipient_id, false);
return redirect()->back();
}
UPDATE!
According to #Ariful. I can add the instance of Auth::user(); to get the id. But it doesn't return me the id to my pivot table and gives me a error.
SQLSTATE[23000]: Integrity constraint violation: 1452 Cannot add or update a child row: a foreign key constraint fails (webdev.document_user, CONSTRAINT document_user_user_id_foreign FOREIGN KEY (user_id) REFERENCES users (id) ON DELETE CASCADE) (SQL: insert into document_user (document_id, user_id) values (59, 0))
public function postDocuments(Request $request)
{
$this->validate($request,
[
'title' => 'required|alpha_dash|max:255',
'content' => 'required',
'category_id' => 'required',
'recipient_id' => 'required',
]);
$user = Auth::user();
$document = new Document();
//Request in the form
$document->title = $request->title;
$document->content = $request->content;
$document->category_id = $request->category_id;
$document->save();
$document->recipients()->sync([$request->recipient_id, $user->id, false]);
return redirect()->back();
}
Models:
User Model
class User extends Model implements AuthenticatableContract
{
public function documents()
{
return $this->belongsToMany('App\Models\Document', 'document_user', 'user_id', 'document_id');
}
}
Document Model:
class Document extends Model
{
public function recipients()
{
return $this->belongsToMany('App\Models\User', 'document_user', 'document_id', 'user_id');
}
}
Migration:
User migration
public function up()
{
Schema::create('users', function (Blueprint $table) {
$table->increments('id');
$table->string('first_name');
$table->string('last_name');
$table->string('middle_name');
$table->string('email');
$table->string('username');
$table->string('address');
$table->string('password');
});
}
Documents migration:
public function up()
{
Schema::create('documents', function (Blueprint $table) {
$table->increments('id');
$table->string('title');
$table->text('content');
$table->integer('category_id')->unsigned();
$table->foreign('category_id')->references('id')->on('categories')->onDelete('cascade');
$table->timestamps();
});
}
documents_user migration:
public function up()
{
Schema::create('document_user',function (Blueprint $table)
{
$table->increments('id');
$table->integer('user_id')->unsigned();
$table->integer('document_id')->unsigned();
$table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
$table->foreign('document_id')->references('id')->on('documents')->onDelete('cascade');
$table->unsignedInteger('sender_id')->nullable();
$table->foreign('sender_id')->references('id')->on('users')->onDelete('cascade');
$table->dateTime('dateReceived')->default(DB::raw('CURRENT_TIMESTAMP'));
});
}
Screenshot Database:
UPDATE 2:
I can insert a values on my user_id based on the user's choice in the select list.
This is where the values of the form inserted in the user_id column. I just need to insert the current user in my sender_id so I can determined who send the data.
<div class = "form-group">
<label for = "recipient_id" class = "control-label">To:</label>
<select name = "recipient_id[]" multiple class = "form-control" id = "myUserList">
#foreach ($resultRecipient as $list)
<option value = "{{ $list->id }}">{{ $list->username }}</option>
#endforeach
</select>
</div>
As you can see here I just insert this manually based on the users table data. Still don't have idea how can I insert the current user into sender_id column.
I believe this should work
$user = Auth::user(); //get current user
$document->recipients()->sync([$user->id]);
UPDATED Source Link
$document->recipients()->sync( [ $request->recipient_id, $user->id ], false );
As per documentation,
The sync method accepts an array of IDs to place on the intermediate table.
UPDATE 2
$document->recipients()->sync( [ $request->recipient_id =>
['sender_id' => $user->id] ],
false );
Your sender_id is not part of your relationship. So you need to add it as extra info.
UPDATE 3
After discussion, this should be your main code
foreach($request->recipient_id as $receipentId){
$document->recipients()->sync( [ $receipentId =>
['sender_id' => $user->id] ],
false );
}
This will loop through your receipent_id array and take each id for sync with the current logged in user as $user->id;

Categories