Hi,

I found a problem that field translation page shows 403.
Attach the patch with fix.

It can be reproduced by steps below.

  1. Install Drupal(8.2.5) with 'Standard' profile
  2. Enable module 'Configuration Translation' and 'Telephone Validation'
  3. Add language in /admin/config/regional/language
  4. Access /admin/structure/types/manage/page/fields
  5. Select 'Translate' link in operations column for Body field
CommentFileSizeAuthor
#3 field_translation_page-2898171-3.patch397 byteszaporylie
fix_field_translation_page_403.patch396 byteshayashi

Comments

hayashi created an issue. See original summary.

zaporylie’s picture

zaporylie
he/him
Primary language Polish
commented
Status: Active » Needs work

Thanks for catching this issue and reporting here. I was in fact able to reproduce this issue.

+++ b/config/schema/telephone_validation.schema.yml
@@ -1,4 +1,4 @@
+field.widget.third_party.telephone_validation:

Third party settings are stored on field, not a widget. I think something like field.field.*.third_party.telephone_validation would suit better here. What do you think?

zaporylie’s picture

zaporylie
he/him
Primary language Polish
commented
Status: Needs work » Needs review
StatusFileSize
new field_translation_page-2898171-3.patch397 bytes

Something like this

  • zaporylie committed 68d123f on 8.x-2.x 
    Issue #2898171 by hayashi, zaporylie: Field translation page shows 403
zaporylie’s picture

zaporylie
he/him
Primary language Polish
commented
Status: Needs review » Fixed 
+++ b/config/schema/telephone_validation.schema.yml
@@ -1,4 +1,4 @@
+field.field.*.third_party.telephone_validation:

Changed to field.field.*.*.*.third_party.telephone_validation on commit.

hayashi’s picture

hayashi commented

I think you are right. Thanks for the fix.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.