Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The language string on my site is en-gb. This is 5 characters so does not fit in the 2 character limit of the language field in the db table push_notifications_tokens. Is the language string an Apple defined value?
Comment | File | Size | Author |
---|---|---|---|
#2 | push_notifications-allow_longer_language_string-1762184-2.patch | 1.28 KB | johnennew |
Comments
Comment #1
haagendazs CreditAttribution: haagendazs commentedArgh. Thanks for catching that. I am getting the list of languages from the _locale_get_predefined_list function, and somehow assumed that all language codes are 2 characters. I'll provide a patch for this shortly.
Comment #2
johnennew CreditAttribution: johnennew commentedHi,
Here is a patch suggestion for review for this issue. The longest langage code returned by _locale_get_predefined_list is 11 characters so it extends the language field to this.
Kind regards,
John
Comment #3
haagendazs CreditAttribution: haagendazs commentedSorry for the late reply about this. I modified your patch slightly and will release this shortly.
Comment #4
haagendazs CreditAttribution: haagendazs commented