Thanks for this great module.
While using this to convert Chinese into Pinyin, I found some bugs.
Here is the patch.
I would like also to request a co-maintainer for this module. To fix the inssues and mainly the Chinese problem.
As there are thousand of words on Chinese and we do need time check to fix them. If I were a co-maintainer , I can fix them directly.
Thanks for your attention.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | transliteration-chinesebug-1998782-8.patch | 1.24 KB | edxxu |
Comments
Comment #1
rogical commentedthe default translation does not working well, it's good to have someone to improve this.
Comment #2
lcnphp commentedGreat Job!
Comment #3
jackniu commentedThis problem has troubled me for a long time, thanks!
Good job!
Comment #4
zterry95 commentedI would like to request an co-maintainer for this module.
The last commits of this module is 10 months ago.
https://drupal.org/node/199246/commits
Also my patch in this tickets get no response more than 2 weeks.
I will help in fix the mistakes in Chinese and other problem in the inssue list.
Thanks for your attention.
Comment #5
dddave commentedI'm sending this back into the module queue so that this issue can still be found there and so that people can actually review the patch.
I would recommend to file a new issue in Transliterations queue requesting co-maintainership and contacting the maintainers of the module directly per mail. After two weeks with no response to that effort you can send that issue into the webmaster queue.
Comment #6
amateescu commentedCommitted and pushed to 7.x-3.x and 6.x-3.x. Thanks and sorry for not replying sooner!
http://drupalcode.org/project/transliteration.git/commit/f44ee0a
http://drupalcode.org/project/transliteration.git/commit/31c0d96
Comment #8
edxxu commentedI found a bug when transliterate simplifed chinese character '惊' into Pinyin. The result should be 'Jing' but 'Liang' instead.
Here is the patch, please review.
Comment #9
edxxu commentedHere is the '惊' item in Chinese dictionary.
http://dict.baidu.com/s?wd=%E6%83%8A
Comment #10
chris matthews commented@edxxu, since this issue has already been committed and pushed to 7.x-3.x and 6.x-3.x, you'll need to open a new issue for #8.