Edit Attribute -> Add Option or Swatch Item Error 404

otheriz

As an additional, this add option problem not occurred when use add new attribute..
Only on edit..

please verify if this is the cause

this is the form data send

some how..it always call/trigger ....->update() instead of ...->create() because it never find any "option_" value...

as it is new option id..when the update triggered..it's not found any existing data..and somehow it throws an error 404 page not found...

when i disable those line..it worked..and of course, the option item not added(because i disabled the script)

otheriz

i Found suspicious form data..
when i see the form data on bagisto live demo..the output is

while on my form data is

otheriz

I have found the way to regenerate this problem...even from live demo..
just try to add Indonesia language

then try to add option item on "Catalog -> attributes -> any item with type 'select'" then save..
same problem

form data

Vaishali Agarwal

Hi @otheriz
We appreciated your efforts to finally debugging the issue.
I've also able to reproduce this issue.

You could raise the issue on github community https://github.com/bagisto/bagisto/issues

Thanks.

otheriz

i will post it on the issue..somehow..i try several country..
it only happen on my country code "id"..
i guess because it same as script "$id"..

---___---

Vaishali Agarwal

@otheriz
That could also be the reason behind this issue.
As i have checked the indonesia locale code is in_ID

Please once try this https://prnt.sc/sgtjth, it might resolve your issue.

otheriz

it's better to use 3 letter ISO code (idn) other than local code..
but it too much to update it..since i already create much translation..

so the other workaround for this issue is i updated this line..
in short:
in this file https://github.com/bagisto/bagisto/blob/master/packages/Webkul/Admin/src/Resources/views/catalog/attributes/edit.blade.php
Change all string 'id' with other than 'id' like 'getID' or whatever..
then replace all variable ".id" with ".getID" (whatever you change)

from

let row = {'id': id};

to (other than "id")

let row = {'getID': id};

and change all "row.id" variable after those line to "row.getID"
Specifically these lines
Line 485,Line 495,Line 499, and Line 503

from

'id': '{{ $option->id }}'

to (same as above changing)

'getID': '{{ $option->id }}'

and change also "row.id" at optionRows inside tag "options-template" to "row.getID"

Specifically this two lines
Line 364 and Line 369

and the variable below that refer to it at :
Line 522
from "optionRow.id" to "optionRow.getID"

and it worked perfectly

Vaishali Agarwal

@otheriz
we will check the feasibility to use ISO Code instead of locale code.
Thanks for the suggestion.

Vaishali Agarwal

@otheriz
this query is resolved on github community raised by you, please check it here https://github.com/bagisto/bagisto/issues/3070

Do let us know if still any doubt.
Thanks

otheriz

I tried with your updated code, and pull from the master..it's worked perfectly now..

thank you for the fixed.

Vaishali Agarwal

Hi @otheriz
Glad to hear that your issue is being resolved!!

Thanks