isAlnum function is incorrect in mailmerge example: check only latin symbols
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 50/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- backend
Direzione di ricerca
Apri gmail/mailmerge/mailmerge.gs e analizza la funzione isAlnum a cui fa riferimento l’issue. Esegui l’esempio di mailmerge con intestazioni di colonna in russo o coreano, quindi verifica che il controllo dei caratteri aggiornato sostituisca quei valori delle intestazioni mantenendo il comportamento esistente per le intestazioni latine.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Expected Behavior
Provided sample mail_merge works as described. Really it works only for headers with latin symbols (e.g. English).
Sample URL: https://sites.google.com/site/scriptsexamples/custom-methods/create-text-from-template#TOC-Documentation
Description: The cause is isAlnum function. That is naively implemented as:
function isAlnum(char) {
return char >= 'A' && char <= 'Z' ||
char >= 'a' && char <= 'z' ||
isDigit(char);
}
But Unicode standard describe much more "Alpha" symbols in different categories.
So really that function should be implemented like:
function isAlnum(char) {
return !! char.match(/[\p{Number}\p{Letter}]/u)
}
and also function isDigit may be dropped (that also potentially has similar problems)
Actual Behavior
Headers on said in Russian or Korean language just ignored and values does not substituted in template.
Steps to Reproduce the Problem
- Run example with column name, said in Russian or Korean language.
- Lingua principale
- JavaScript
- Stelle
- 5.2k
- Fork
- 2k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di googleworkspace/apps-script-samples
-
Alvin Litana MirasolAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 1/100
googleworkspace/apps-script-samples#659 · 1 commento · 1 reazione ·
-
Password resetAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
googleworkspace/apps-script-samples#654 · 2 reazioni ·
-
cod gsAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
googleworkspace/apps-script-samples#651 · 1 reazione ·
-
tespogromeAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 1/100
googleworkspace/apps-script-samples#624 · 1 reazione ·
-
Get_order_manually.gsForse di nuovo libera @vinay-google l’ha presa 242 giorni fa e non c’è nessuna pull request aperta. Aperta
googleworkspace/apps-script-samples#616 · 1 assegnatario ·
Tutte le issue di googleworkspace/apps-script-samples
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
521xueweihan/HelloGitHub#3939 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
needs-website-changes
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
[Bug] critique requires the detector on native targets, but audit.native.md says it does not applyApertaneeds triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
pbakaus/impeccable#1017 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno