See title, #1801962: [meta] Convert custom non-volatile cache-alike things to k/v store and #1627006: Collect project information for translation update.

CommentFileSizeAuthor
#4 remote-cache-locale-1826294-3.patch993 bytesBerdir
#1 remove-cache-locale-1826294-1.patch24.87 KBBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
FileSize
remove-cache-locale-1826294-1.patch24.87 KB

Looks like we're not using this at all?

Maybe this was supposed to be used in #1804688: Download and import interface translations but that issue already switched to using state().

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Title: Convert custom cache_update-alike cache in locale.module to expriable k/v store » Remove cache_locale

Updating title, as there's nothing that needs to be replaced currently :)

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Reviewed & tested by the community

Hm, messed that patch up. Here's a correct one.

@Sutharsan confirmed in #1804688-15: Download and import interface translations that this isn't necessary. Who wants to RTBC? :)

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Reviewed & tested by the community » Needs review
FileSize
remote-cache-locale-1826294-3.patch993 bytes

Uh. Not what I wanted to do ;)

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +D8MI, +language-ui

Looks good to me.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Fix link.