Edit Attribute -> Add Option or Swatch Item Error 404

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