This patch enable translation of validation error messages (addViolation).

CommentFileSizeAuthor
#4 node_title_validation-translate-messages-2964665-3-D8.patch5.79 KBKrzysztof Domański
#4 node_title_validation-translate-messages.JPG25.98 KBKrzysztof Domański
translations.patch2.56 KBdgaspara
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dgaspara created an issue. See original summary.

jcisio’s picture

jcisio CreditAttribution: jcisio as a volunteer and at Axess Open Web Services for Institut français commented
Version: 8.x-1.1 » 8.x-1.x-dev
Status: Active » Needs work

We need to use dependency injection of the translation service.

mahtab_alam’s picture

mahtab_alam CreditAttribution: mahtab_alam at Valuebound commented

$this->t is not working here. Throwing error as Error: Call to undefined method Drupal\\node_title_validation\\Plugin\\Validation\\Constraint\\NodeTitleConstraintValidator::t()

Krzysztof Domański’s picture

Krzysztof Domański
Location Poland
CreditAttribution: Krzysztof Domański as a volunteer commented
Status: Needs work » Needs review
FileSize
node_title_validation-translate-messages.JPG25.98 KB
node_title_validation-translate-messages-2964665-3-D8.patch5.79 KB

I use dependency injection of the translation service. Now we can translate all error messages.

dgaspara’s picture

dgaspara CreditAttribution: dgaspara at NTT DATA commented
dgaspara’s picture

dgaspara CreditAttribution: dgaspara at NTT DATA commented
dgaspara’s picture

dgaspara CreditAttribution: dgaspara at NTT DATA commented
dgaspara’s picture

dgaspara CreditAttribution: dgaspara at NTT DATA commented

Status: Needs review » Needs work

The last submitted patch, 4: node_title_validation-translate-messages-2964665-3-D8.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Renrhaf’s picture

Renrhaf
Primary language French
Location 📍 Strasbourg 🐦🦜
CreditAttribution: Renrhaf at CGI for Ville de Montréal commented

Tested in our current site, everything works properly with the patch in #4

Renrhaf’s picture

Renrhaf
Primary language French
Location 📍 Strasbourg 🐦🦜
CreditAttribution: Renrhaf at CGI for Ville de Montréal commented
Status: Needs work » Reviewed & tested by the community

cgomezg’s picture

cgomezg
Location Montreal
CreditAttribution: cgomezg as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Merged in 8.x-1.x-dev

Status: Fixed » Closed (fixed)

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