isAlnum function is incorrect in mailmerge example: check only latin symbols
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 50/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- backend
Hướng nghiên cứu
Mở gmail/mailmerge/mailmerge.gs và kiểm tra hàm isAlnum được tham chiếu trong issue. Chạy ví dụ mailmerge với tiêu đề cột bằng tiếng Nga hoặc tiếng Hàn, sau đó xác minh rằng việc kiểm tra ký tự đã cập nhật thay thế các giá trị tiêu đề đó đồng thời giữ nguyên hành vi hiện có đối với các tiêu đề bằng chữ Latin.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- JavaScript
- Star
- 5.2k
- Fork
- 2k
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của googleworkspace/apps-script-samples
-
Alvin Litana MirasolĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 1/100
googleworkspace/apps-script-samples#659 · 1 bình luận · 1 reaction ·
-
Password resetĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
googleworkspace/apps-script-samples#654 · 2 reaction ·
-
cod gsĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 10/100
googleworkspace/apps-script-samples#651 · 1 reaction ·
-
tespogromeĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 1/100
googleworkspace/apps-script-samples#624 · 1 reaction ·
-
Get_order_manually.gsCó thể làm lại được @vinay-google đã nhận 232 ngày trước và không có pull request nào đang mở. Đang mở
googleworkspace/apps-script-samples#616 · 1 người được giao ·
Tất cả issue của googleworkspace/apps-script-samples
Issue tương tự
-
Daemon delete dialog: "Remove all ExApps" checkbox and `removeExApps` parameter have no effectĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Maintainer thường phản hồi trong vòng 3 ngày
-
Add: Atlas TVĐang mởchannels:add check:passed
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
Maintainer thường phản hồi trong vòng 4 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
jaegertracing/jaeger-ui#4547 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
feedback simulation workshop
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
githubnext/gh-aw-workshop#4090 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug deck: add to staging level: missing p-feature: Manage Submissions p-feature: Submissions and process priority: MUST HAVE ready for dev lead role: missing size: missing time sensitive
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
hackforla/tdm-calculator#3581 ·
Maintainer thường phản hồi trong vòng 2 ngày