Problem/Motivation

Varies issues in shortcut schema.

Proposed resolution

1. Fix schema to support all shortcut. i.e. changing shortcut.set.defult to shortcut.set.*
2. Add additional fields in active config to schema - missing links and status

Remaining tasks

#1 is fixed as part of #1964254: Configuration schemas missing langcode and uuid at places need to fix #2.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vijaycs85’s picture

Status: Active » Needs review
FileSize
866 bytes

Fixing #2 here... #3 seems needs some code re-factoring which I will confirm with @alexpott or @Gabor before start.

vijaycs85’s picture

minor issue in schema file.

Gábor Hojtsy’s picture

Status: Needs review » Postponed
Issue tags: +sprint

Let's have this postponed on #1964254: Configuration schemas missing langcode and uuid at places, since that is RTBC and heavily overlaps with this.

vijaycs85’s picture

Status: Postponed » Needs review
Gábor Hojtsy’s picture

Status: Needs review » Needs work

Above patch should not apply anymore due to #1964254: Configuration schemas missing langcode and uuid at places committed.

vijaycs85’s picture

Status: Needs work » Needs review
FileSize
639 bytes

Re-rolling...

vijaycs85’s picture

FileSize
5.77 KB

Adding screenshot for issue #3 in summary.

Gábor Hojtsy’s picture

Status: Needs review » Needs work
+++ b/core/modules/shortcut/config/schema/shortcut.schema.ymlundefined
@@ -16,3 +16,15 @@ shortcut.set.*:
     langcode:
       type: string
       label: 'Default language'
...
+    langcode:
+      type: string

Langcode is already there?! #1964254: Configuration schemas missing langcode and uuid at places already added langcode. Not status, but langcode it did :)

vijaycs85’s picture

Thanks @Gábor Hojtsy. Removed duplicate langcode and added changes for #3 in summary. Links will be saved as array now.

id: content-author
uuid: d248406d-ad5c-4da8-b98e-75b8026a4285
label: 'Content author'
links:
  - 867177af-3542-4ea5-9923-f75dbcbaf081
  - ee575e60-9d75-4637-8785-03c6b854bec2
  - 35fdd99d-b405-43dd-8f3e-7f65853024bb
status: '1'
langcode: en

UPDATE: locally throwing some error on adding new shortcut set. This needs more work...

vijaycs85’s picture

Status: Needs work » Needs review
FileSize
1.82 KB

Attaching patch for #9 :)

Gábor Hojtsy’s picture

Do we *need* to modify the format here so we can write a schema for it, or is just a choice we can make?

Gábor Hojtsy’s picture

In other words, I'd avoid modifying the format if at all possible and just document it in this issue.

Gábor Hojtsy’s picture

Issue summary: View changes

Updated issue summary.

vijaycs85’s picture

Moving the format problem as a separate issue #1971456: Make usage of shortcut.settings.links consistent. This change doesn't affect schema, so this issue is not blocked :)

vijaycs85’s picture

Attaching inspector form with minor label fix.

1964588-14.png

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

Looks like all good, thanks.

Status: Reviewed & tested by the community » Needs work
Issue tags: -Quick fix, -Configuration system, -D8MI, -sprint, -language-config, -Configuration schema

The last submitted patch, 1964588-shortcut-config-schema-14.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 1964588-shortcut-config-schema-14.patch, failed testing.

vijaycs85’s picture

Status: Needs work » Needs review
Issue tags: +Quick fix, +Configuration system, +D8MI, +sprint, +language-config, +Configuration schema
Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Back to RTBC as per #15. Test fails were unrelated.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Gábor Hojtsy’s picture

Issue tags: -sprint

Superb, thanks!

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

Anonymous’s picture

Issue summary: View changes

Updated issue summary.