АК
Почему дублировать? Из Url просто выпилить эту возможность
Size: a a a
2020 May 26
C
речь только о том, что в Url атрибуты не нужны, от слова совсем
C
Я предполагаю, что это пошло от меню, где url хранится в базе с атрибутами
AP
Нет, не оттуда) https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Url.php/function/Url%3A%3AfromUri/8.8.x отлично описаны опции, и если их разделить, как вы предлагаете, то придется их дублировать во множестве мест и DX----, но количество ошибок++
AP
Если вернуться на 8 лет назад, то это было самым горячим обсуждением - стоит ли разделять linkGenerator и urlGenerator, и как в д7 прикрутить понятие роутов)
C
ничего дублировать не придётся, атрибуты переедут из url object в link object
АК
Нет, не оттуда) https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Url.php/function/Url%3A%3AfromUri/8.8.x отлично описаны опции, и если их разделить, как вы предлагаете, то придется их дублировать во множестве мест и DX----, но количество ошибок++
Любой костыль рождается только потому, что так сделать проще))
АК
Почему в опенкарт нет ORM? Да потому что "придется их дублировать во множестве мест и DX----, но количество ошибок++"
AP
Алексей Кузнецов
Любой костыль рождается только потому, что так сделать проще))
Посмотри на 3года обсуждений и сколько комитов таскали код из одного места в другое и "проще" явно не об этом
AP
ничего дублировать не придётся, атрибуты переедут из url object в link object
Давай сделаем иш, и ты попробуешь какой-нибудь модуль конвертировать, и дифы всё покажут
C
Тут не модуль, а ядро надо патчить
C
Для начала можно просто разрешить хранить атрибуты в Link, а в Url сделать deprecated
АК
В любом случае, в урл нет всех параметров, по которым можно сгенерировать линк, поэтому просто объявление параметров переедет из одного места в другое, а ничего дополнительного делать не придётся
AP
https://www.drupal.org/project/drupal/issues/2385373 можно разморозить
AP
Алексей Кузнецов
В любом случае, в урл нет всех параметров, по которым можно сгенерировать линк, поэтому просто объявление параметров переедет из одного места в другое, а ничего дополнительного делать не придётся
Не совсем так, но можешь попробовать патч сделать, с удовольствием помогу
АК
Надо создать ишью, а там Алёны набегут и пофиксят
АК
С их техническим бэкграундом это не составит труда
AP
Алексей Кузнецов
Надо создать ишью, а там Алёны набегут и пофиксят
Я же выше ссыль кинул - уже создано
АК
ну там обсуждают депрекейтед метод вообще-то
АК
Нужно провести инвестигейшн, структурировать и подать инфу в актуальном виде