Replace slash chars in all “key/value” phrases to a hyphen

Created on 17 July 2024, over 1 year ago

Updated 23 July 2024, over 1 year ago

Problem/Motivation

Releasing a new version with D11 compatibility can be a good time for such a linguistic change I suggest for consideration to this module. Stop using the slash sign (“/”) between the “key” and “value” words in all user-facing contexts. This character carries the general meaning of “OR”. In our everyday written language it's being used with the meaning “choose this one / or that one but not both” usually.

Proposed resolution

My suggestion is to shift towards using a hyphen (“-”) character instead. Hyphen has the global meaning of connecting two separate terms into one word. To be more exact regarding the IT jargon, Wikipedia also refers to such data structures as “key-value”.

Remaining tasks

Run a search & replace on the module codebase with these regex-patterns:

Search for: Key\s?\/\s?(V|v)alue → Replace to: “Key-value”
Search for: key\s?\/\s?(V|v)alue → Replace to: “key-value”

User interface changes

More natural language easier for everyone to grasp.

📌 Task

Status

RTBC

Version

1.0

Component

User interface

Created by

🇭🇺Hungary Balu Ertl Budapest 🇪🇺

Live updates comments and jobs are added and updated live.

Novice
It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Merge Requests

!10Replace slash chars in all “key/value” phrases to a hyphen
Open
🇮🇳India bhaveshdas
updated over 1 year ago

Comments & Activities

Issue created by @Balu Ertl
First commit to issue fork.
Merge request !103462097-replace-slash-chars:remove slash and add hyphen → (Open) created by bhaveshdas
Status changed to Needs review over 1 year ago2:03pm 23 July 2024
Comment over 1 year ago →
🇮🇳India bhaveshdas
Hey, I have replace slash to hyphen as per suggestion.
Could you please review it.

Thankyou !!
Status changed to RTBC over 1 year ago4:53pm 23 July 2024
Comment over 1 year ago →
🇭🇺Hungary Balu Ertl Budapest 🇪🇺
Thanks @bhaveshdas → for the quick solution. I grepped the codebase for the combined regex pattern (K|k)ey\s?\/\s?(V|v)alue with no results → all occurrences has been changed to their case-sensitive replacement. Also, I tested the module and it works as expected:

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024