Vietnamese slug cho Discourse

Nhưng mà pull request hoài mà không chấp nhận

Jeff không đồng ý với slug này đâu đâu. @thangngoc89 phải thay vào

 if ['zh_CN', 'ja'].include?(SiteSetting.default_locale)

thành

 if ['zh_CN', 'ja', 'vi'].include?(SiteSetting.default_locale)

Nếu stringex có 'vi'

Cái này là tuyệt đối không được.

#For Vietnamese slug 
vietnamese   = "àáạảãâầấậẩẫăằắặẳẵèéẹẻẽêềếệểễìíịỉĩòóọỏõôồốộổỗơờớợởỡùúụủũưừứựửữỳýỵỷỹđÀÁẠẢÃÂẦẤẬẨẪĂẰẮẶẲẴÈÉẸẺẼÊỀẾỆỂỄÌÍỊỈĨÒÓỌỎÕÔỒỐỘỔỖƠỜỚỢỞỠÙÚỤỦŨƯỪỨỰỬỮỲÝỴỶỸĐêùà"
replacements = "aaaaaaaaaaaaaaaaaeeeeeeeeeeeiiiiiooooooooooooooooouuuuuuuuuuuyyyyydAAAAAAAAAAAAAAAAAEEEEEEEEEEEIIIIIOOOOOOOOOOOOOOOOOUUUUUUUUUUUYYYYYDeua"
string = string.tr(vietnamese, replacements)
#End Vietnamese slug

cái này không phải là chức năng trong core nên reject là hoàn toàn dễ hiểu mà.

bạn phát triển riêng thành module plugin là được

thêm cả code style của bạn hardcode hơi quá, nhét cả text unicode vô trong string nữa là không được.

thiếu cả không có tests suite.

Ngày trước Jeff không đồng ý, nhưng sau này thì họ đồng ý slug cho các ngôn ngữ khác tiếng anh.

Nhưng theo post của @thangngoc89 trên meta.discourse.org thì em nghĩ là có giải pháp. Với điều kiện mình khai báo cho stringex

Do em không có rành về Ruby (chỉ nghiên cứu khi đến với Discourse) nên không rõ nữa.
@ltd anh có thể tiếp tục nghiên cứu không nhỉ?

@laptrinhio trong php mình có contribute vào cái Slugify. Bây giờ làm project nào thì chỉ cần require nó là được. Ruby thì mình mù tịt

Cái stringex vi này có vẻ không ổn lắm:

thangngoc89/stringex/blob/7917085cee35f9a14b4782f48eb1a2d5cd0a73bf/locales/vi.yml

# NOTE: Some translation keys make use of matches from regular expressions
# to manipulate whitespace and order. Please consult the source code for
# Stringex::Localization::ConversionExpressions to see what those
# regular expressions look like if you need to manipulate the order
# differently than the usage below.
vi:
  stringex:
    characters:
      and: va
      at: tai
      divide: divided by
      degrees: do
      dot: \1 cham \2
      ellipsis: dot dot dot
      equals: bang
      number: number
      percent: phan tram
      plus: cong
      slash: slash
      star: sao

Đạt cũng không rành Ruby, nhưng mình phải giải quyết theo cách sử dụng stringex cho nó gọn code chứ không thể làm 2 cái mảng như thế này được

#For Vietnamese slug 
vietnamese   = "àáạảãâầấậẩẫăằắặẳẵèéẹẻẽêềếệểễìíịỉĩòóọỏõôồốộổỗơờớợởỡùúụủũưừứựửữỳýỵỷỹđÀÁẠẢÃÂẦẤẬẨẪĂẰẮẶẲẴÈÉẸẺẼÊỀẾỆỂỄÌÍỊỈĨÒÓỌỎÕÔỒỐỘỔỖƠỜỚỢỞỠÙÚỤỦŨƯỪỨỰỬỮỲÝỴỶỸĐêùà"
replacements = "aaaaaaaaaaaaaaaaaeeeeeeeeeeeiiiiiooooooooooooooooouuuuuuuuuuuyyyyydAAAAAAAAAAAAAAAAAEEEEEEEEEEEIIIIIOOOOOOOOOOOOOOOOOUUUUUUUUUUUYYYYYDeua"
string = string.tr(vietnamese, replacements)
#End Vietnamese slug

P/S: Đã star cái này lại và nghiên cứu

@ltd cái Stringex đúng là rất không ổn.
Ngó phát qua php thì nó sẽ ra thế này

cocur/slugify/blob/master/src/Slugify.php#L444-534

<?php


/**
* This file is part of cocur/slugify.
*
* (c) Florian Eckerstorfer <florian@eckerstorfer.co>
*
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/


namespace Cocur\Slugify;


use Cocur\Slugify\RuleProvider\DefaultRuleProvider;
use Cocur\Slugify\RuleProvider\RuleProviderInterface;


/**
* Slugify
*
* @package   Cocur\Slugify
* @author    Florian Eckerstorfer <florian@eckerstorfer.co>
* @author    Ivo Bathke <ivo.bathke@gmail.com>
* @author    Marchenko Alexandr
* @copyright 2012-2015 Florian Eckerstorfer
* @license   http://www.opensource.org/licenses/MIT The MIT License
*/
class Slugify implements SlugifyInterface
{
   const LOWERCASE_NUMBERS_DASHES = '/([^A-Za-z0-9]|-)+/';


   /**
    * @var array<string,string>
    */
   protected $rules = [];


   /**
    * @var RuleProviderInterface
    */
   protected $provider;


   /**
    * @var array<string,mixed>
    */
   protected $options = [
       'regexp'    => self::LOWERCASE_NUMBERS_DASHES,
       'separator' => '-',
       'lowercase' => true,
       'trim' => true,
       'strip_tags' => false,
       'rulesets'  => [
           'default',
           // Languages are preferred if they appear later, list is ordered by number of
           // websites in that language
           // https://en.wikipedia.org/wiki/Languages_used_on_the_Internet#Content_languages_for_websites
           'azerbaijani',
           'burmese',
           'hindi',
           'georgian',
           'norwegian',
           'vietnamese',
           'ukrainian',
           'latvian',
           'finnish',
           'greek',
           'czech',
           'arabic',
           'turkish',
           'polish',
           'german',
           'russian',
           'romanian'
       ],
   ];


   /**
    * @param array                 $options
    * @param RuleProviderInterface $provider
    */
   public function __construct(array $options = [], RuleProviderInterface $provider = null)
   {
       $this->options  = array_merge($this->options, $options);
       $this->provider = $provider ? $provider : new DefaultRuleProvider();


       foreach ($this->options['rulesets'] as $ruleSet) {
           $this->activateRuleSet($ruleSet);
       }
   }


   /**
    * Returns the slug-version of the string.
    *
    * @param string            $string  String to slugify
    * @param string|array|null $options Options
    *
    * @return string Slugified version of the string
    */
   public function slugify($string, $options = null)
   {
       // BC: the second argument used to be the separator
       if (is_string($options)) {
           $separator            = $options;
           $options              = [];
           $options['separator'] = $separator;
       }


       $options = array_merge($this->options, (array) $options);


       // Add a custom ruleset without touching the default rules
       if (isset($options['ruleset'])) {
           $rules = array_merge($this->rules, $this->provider->getRules($options['ruleset']));
       } else {
           $rules = $this->rules;
       }


       $string = ($options['strip_tags'])
           ? strip_tags($string)
           : $string;


       $string = strtr($string, $rules);
       unset($rules);


       if ($options['lowercase']) {
           $string = mb_strtolower($string);
       }


       $string = preg_replace($options['regexp'], $options['separator'], $string);


       return ($options['trim'])
           ? trim($string, $options['separator'])
           : $string;
   }


   /**
    * Adds a custom rule to Slugify.
    *
    * @param string $character   Character
    * @param string $replacement Replacement character
    *
    * @return Slugify
    */
   public function addRule($character, $replacement)
   {
       $this->rules[$character] = $replacement;


       return $this;
   }


   /**
    * Adds multiple rules to Slugify.
    *
    * @param array <string,string> $rules
    *
    * @return Slugify
    */
   public function addRules(array $rules)
   {
       foreach ($rules as $character => $replacement) {
           $this->addRule($character, $replacement);
       }


       return $this;
   }


   /**
    * @param string $ruleSet
    *
    * @return Slugify
    */
   public function activateRuleSet($ruleSet)
   {
       return $this->addRules($this->provider->getRules($ruleSet));
   }


   /**
    * Static method to create new instance of {@see Slugify}.
    *
    * @param array <string,mixed> $options
    *
    * @return Slugify
    */
   public static function create(array $options = [])
   {
       return new static($options);
   }
}

cái Stringex đúng là rất không ổn.

Không, @thangngoc89 hiểu sai vấn đề rồi, cách họ translate vẫn rất ổn.

rsl/stringex/blob/39866d8235b757dcbcd80761f2150a799f8aa020/locales/pl.yml#L71-L88

Ą: A
Ć: C
Ę: E
Ł: L
Ń: N
Ó: O
Ś: S
Ź: Z
Ż: Z
ą: a
ć: c
ę: e
ł: l
ń: n
ó: o
ś: s
ź: z
ż: z

Trước hết mình phải pull request cho stringex vi trước, Tuy nhiên cái commit cho stringex vi này chưa được ổn

thangngoc89/stringex/blob/7917085cee35f9a14b4782f48eb1a2d5cd0a73bf/locales/vi.yml

# NOTE: Some translation keys make use of matches from regular expressions
# to manipulate whitespace and order. Please consult the source code for
# Stringex::Localization::ConversionExpressions to see what those
# regular expressions look like if you need to manipulate the order
# differently than the usage below.
vi:
  stringex:
    characters:
      and: va
      at: tai
      divide: divided by
      degrees: do
      dot: \1 cham \2
      ellipsis: dot dot dot
      equals: bang
      number: number
      percent: phan tram
      plus: cong
      slash: slash
      star: sao

Và mình thiếu test cho cái stringex vi này

hey looks good. do you mind writing a test file for this? you can find examples in the test/unit/localization directory. need to have something to test against regressions for this. thanks.

@ltd vấn đề là nằm ở chỗ default locale của Discourse. Mình không thể setting Vietnamese lên làm mặc định được :
https://www.transifex.com/projects/p/discourse-org/language/vi_VN/

Công việc translate mới được 36% (thời gian gần đây chỉ có mình làm việc translate này)

Test file thì gần như Khoa không biết làm (Ruby noob)

Có hai lý do mà Đạt không làm cái này bởi vì

• Đạt không muốn dịch sang tiếng Việt
• Làm cái này cần đam mê

Về cái slug thì trước đây Đạt cũng nghĩ tới, có thể sẽ làm cái slug này ở local thôi, không nhất thiết phải đưa vào core.

Okie. Đạt. Hy vọng là sẽ có được một giải pháp slug nhẹ nhàng, phù hợp cho Discourse.

@thangngoc89, cái code đầu tiên bạn đưa ra ở trên của bạn cũng có thể apply cho DC local được rồi đấy, không nhất thiết phải PR để đưa vào core.

#For Vietnamese slug 
vietnamese   = "àáạảãâầấậẩẫăằắặẳẵèéẹẻẽêềếệểễìíịỉĩòóọỏõôồốộổỗơờớợởỡùúụủũưừứựửữỳýỵỷỹđÀÁẠẢÃÂẦẤẬẨẪĂẰẮẶẲẴÈÉẸẺẼÊỀẾỆỂỄÌÍỊỈĨÒÓỌỎÕÔỒỐỘỔỖƠỜỚỢỞỠÙÚỤỦŨƯỪỨỰỬỮỲÝỴỶỸĐêùà"
replacements = "aaaaaaaaaaaaaaaaaeeeeeeeeeeeiiiiiooooooooooooooooouuuuuuuuuuuyyyyydAAAAAAAAAAAAAAAAAEEEEEEEEEEEIIIIIOOOOOOOOOOOOOOOOOUUUUUUUUUUUYYYYYDeua"
string = string.tr(vietnamese, replacements)
#End Vietnamese slug

còn nếu muốn vào core thì phải làm như post trên:

Trước hết mình phải pull request cho stringex vi trước, Tuy nhiên cái commit cho stringex vi này chưa được ổn

Okie @ltd mình đang áp dụng nó cho local mà.

@ltd Không có gì mới, chỉ gom nó vào 1 plugin thôi

GitHub

thangngoc89/discourse-vietnamese-slug

Add Vietnamese Slug to Discourse. Contribute to thangngoc89/discourse-vietnamese-slug development by creating an account on GitHub.

Nice, Đạt sẽ install cái này vào, nếu có bug thì Đạt sẽ report

P/S: Đã install rồi nhé @thangngoc89, chạy rất tốt

Còn 1 lỗi mà làm biếng chưa bao giờ làm đó là hỗ trợ unicode tổ hợp (combine unicode)

Ví dụ topic: Hỏi vui: Mọi người hay gọi hai ký tự này là gì: '/' '\'

oh, unicode tổ hợp nó có ký tự khác hả? Vậy còn có unicode nào nữa không?

Chỉ đơn giản là quăng 1 bộ kí tự unicode và là xong

Cái topic kia bị thế có thể là do em dùng bàn phím swype trên android nên nó k giống bình thường

Như @thangngoc89 nói thì unicode tổ hợp bị lỗi. Rất có thể swype cũng sử dụng unicode tổ hợp. Cái này fix cũng dễ thôi. Có điều lười

Để sang Sing mà thấy không có ai làm thì Đạt commit cho bộ unicode tổ hợp.