Zbyt długie linki w postach i mikroblogach nie są skracane #244

[adamwojs]

Zgodnie z treścią zgołoszenia dodano skracanie URLi w przypadku przekroczenia 64 znaków. Zrzut ekranu: https://dl.dropboxusercontent.com/u/435780/Zrzut%20ekranu%20z%202017-01-10%2022-36-12.png

[adam-boduch]

Dzięki za pulll request.

[adam-boduch]

Warunek sprawdzający długość linku, przeniósłbym do metody truncate(). Przykładowo, tak wygląda metoda limit() w laravel (z tym, że ona dodaje ... na końcu tekstu zamiast [...] w środek linku):

    public static function limit($value, $limit = 100, $end = '...')
    {
        if (mb_strwidth($value, 'UTF-8') <= $limit) {
            return $value;
        }

        return rtrim(mb_strimwidth($value, 0, $limit, '', 'UTF-8')).$end;
    }

Czyli jeżeli długość linka jest ok - metoda zwraca niezmodyfikowany tekst.

[adamwojs]

Zaktualizuje dzisiaj wieczorem pull requesta. Zastanawiałem się też czy nie warto przenieść tej metody do Coyote\Services\Parser\Helpers\Link bo jest troche na skraju opowiedzialności Coyote\Services\Parser\Parsers\Autolink ?

[adam-boduch]

Myślałem, że można iść dalej i połączyć te dwie klasy. Wszak obydwie mają za zadanie formatowanie linków na stronie.

Ta klasa Autolink pierwotnie nie była w projekcie a zamianą URL-i na działające linki <a>, zajmował się parsedown, ale ten ma bardzo poważnego buga: erusev/parsedown#432 stąd też musiałem wyłaczyć zamianę linków w parsedown i dodać własny parser :|

[adamwojs]

Poprawione zgodnie z obietnicą.