fix(user, composer): removing martial status, dependencies (fixes #1587) by MrKrasnov · Pull Request #1590 · OpenVK/openvk

MrKrasnov · 2026-05-17T16:27:49Z

feat(cli): generating users

og content

additional:
FIX incorrect php version for composer.json We use php 8.2
Added CLI command for generate test users
Changed placeholder for edit marital status

veselcraft

ощущение что это навайбкожено

veselcraft · 2026-05-18T15:45:56Z

 "your_email_address" = "Adras Vašaj elektronaj pošty";
 "your_page_address" = "Adras Vašaj staronki";
 "page_address" = "Adras staronki";
+"marital_status_user_id" = "ID karystálnika";


не трогайте плиз локали на языках которых вы не разговариваете. этим занимаются переводчики

Мне писать все в латиницу или вообще не писать ключи на другие strings ?

Все языки фолбэчатся на русский или английский. Как следствие, русского и английского достаточно

исчерпывающий ответ @WerySkok

veselcraft · 2026-05-18T15:52:30Z

+                            <input type="text" placeholder="{_marital_status_user_id}" name="maritalstatus-user"
+                                   n:attr="value => $user->getMaritalStatusUser() ? ltrim($user->getMaritalStatusUser()->getURL(), '/') : ''" />


вместо того, чтобы городить костыли с ltrim, можно было создать в объекте User отдельную функцию для screenname

В корне не согласен. Для показа чего либо во вьюхе совсем не повод захломлять методами и без того большую модель. Модель в целом должна использоваться для возрата данных в данном случае а стилизация текста это задача самой вьюхи.

Исключением может быть yii2 😭

Блин 1822 строчки, его бы декомпозировать как-то по хорошему

Рефакторы только после тестов, пожалуйста

В корне не согласен. Для показа чего либо во вьюхе совсем не повод захломлять методами и без того большую модель. Модель в целом должна использоваться для возрата данных в данном случае а стилизация текста это задача самой вьюхи.

Остаюсь на своём - это костыли. В API используется (увы) один уже такой связанный со screen_name (он просто повторяет функцию, без /), и вот ещё один тут. Но вот для отдельной ф-ции я погорячился, можно прост добавить отдельный аргумент в getURL() с параметром типа nobackslash

Это более рациональное решение, добавил b21d088

veselcraft · 2026-05-18T15:57:38Z

алсо где использование Conventional Commits?

MrKrasnov · 2026-05-18T17:13:46Z

"алсо где использование Conventional Commits?"
В работе коммиты оформляю как и в этом ПР, дальнейщие коммиты могу оформлять в соотвествии с "Conventional Commits"

veselcraft · 2026-05-18T19:39:22Z

lint всё ещё не проходит и не поправлены те моменты, на которые я указал

MrKrasnov · 2026-05-18T20:52:34Z

@veselcraft дай ответ #1590 (comment) .
Линтер подправил.

По файлам Edit.latte
UserPresenter.php
дал свой фидбек почему такие изменения не вижу целесообразными.

WerySkok · 2026-05-18T21:04:53Z

    },
    "require": {
-        "php": "~7.3||~8.1",
+        "php": "~7.3||~8.2",


Не видел инстансов, где 7 пыха всё ещё используется, можно смело дропать, но это лучше оставить на другой ПР, ибо придётся тянуть изменения в доках

солидарен, лучше дропнуть отдельным PR. 7.3 и уж тем более 7.4 уже 4 года как EOL, дистры его не предоставляют в пакетах

вернул версию - revert(composer.json): revert php version

Создал отдельный ПР - #1592

veselcraft

по поводу функции не согласен. всё остальное - ок, молодец!

veselcraft · 2026-05-18T22:19:21Z

 "your_email_address" = "Adras Vašaj elektronaj pošty";
 "your_page_address" = "Adras Vašaj staronki";
 "page_address" = "Adras staronki";
+"marital_status_user_id" = "ID karystálnika";


исчерпывающий ответ @WerySkok

veselcraft · 2026-05-18T22:21:38Z

+                            <input type="text" placeholder="{_marital_status_user_id}" name="maritalstatus-user"
+                                   n:attr="value => $user->getMaritalStatusUser() ? ltrim($user->getMaritalStatusUser()->getURL(), '/') : ''" />


В корне не согласен. Для показа чего либо во вьюхе совсем не повод захломлять методами и без того большую модель. Модель в целом должна использоваться для возрата данных в данном случае а стилизация текста это задача самой вьюхи.

Остаюсь на своём - это костыли. В API используется (увы) один уже такой связанный со screen_name (он просто повторяет функцию, без /), и вот ещё один тут. Но вот для отдельной ф-ции я погорячился, можно прост добавить отдельный аргумент в getURL() с параметром типа nobackslash

veselcraft · 2026-05-24T14:44:30Z

@WerySkok верикок проверьте пулл пожлауйтса

WerySkok · 2026-05-24T14:46:57Z

не та ветка 😭

MrKrasnov · 2026-05-25T05:31:16Z

не та ветка 😭

да, я обосрался и откатил изменения 💩

MrKrasnov added 5 commits May 15, 2026 23:40

FIX incorrect php version for composer.json We use php 8.2

17905c0

Added CLI command for generate test users

6c38315

Fixed: Removing marital status user

63dd14d

Changed set last name for cli

8d9ca1f

Changed placeholder for edit marital status

f202690

MrKrasnov mentioned this pull request May 17, 2026

Невозможность сбросить партнёра в статусе семейного положения #1587

Open

veselcraft requested changes May 18, 2026

View reviewed changes

MrKrasnov requested a review from veselcraft May 18, 2026 17:14

build(deps): install fakerphp/faker

950bd92

veselcraft changed the title ~~issues-1587 Fixed: Removing marital status user~~ fix(user, cli, composer): removing martial status, cli for generating users May 18, 2026

veselcraft changed the title ~~fix(user, cli, composer): removing martial status, cli for generating users~~ fix(user, composer): removing martial status, cli for generating users (fixes #1587) May 18, 2026

veselcraft changed the title ~~fix(user, composer): removing martial status, cli for generating users (fixes #1587)~~ fix(user, composer): removing martial status, dependencies (fixes #1587) May 18, 2026

MrKrasnov added 2 commits May 18, 2026 23:13

feat(cli): use Faker to generate user data

02681fb

style(cli): fix php-cs-fixer formatting

cdc741f

WerySkok reviewed May 18, 2026

View reviewed changes

veselcraft requested changes May 18, 2026

View reviewed changes

MrKrasnov added 3 commits May 23, 2026 17:48

revert(composer.json): revert php version

bb53df7

revert (locales): removed locales key "marital_status_user_id"

9377e72

refactor (edit): added param trim backslash for getURL

b21d088

MrKrasnov requested review from WerySkok and veselcraft May 23, 2026 15:30

veselcraft approved these changes May 23, 2026

View reviewed changes

MrKrasnov added 2 commits May 24, 2026 14:22

chore (bootstrap): changed text for error incompatible php version

c14ef60

revert(bootstrap): reverted text for error incompatible php version

17f3ecb

		<input type="text" placeholder="{_marital_status_user_id}" name="maritalstatus-user"
		n:attr="value => $user->getMaritalStatusUser() ? ltrim($user->getMaritalStatusUser()->getURL(), '/') : ''" />

Uh oh!

Conversation

MrKrasnov commented May 17, 2026 • edited by veselcraft Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

og content

Uh oh!

veselcraft left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

veselcraft commented May 18, 2026

Uh oh!

MrKrasnov commented May 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

veselcraft commented May 18, 2026

Uh oh!

MrKrasnov commented May 18, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

veselcraft left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

veselcraft commented May 24, 2026

Uh oh!

WerySkok commented May 24, 2026

Uh oh!

MrKrasnov commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

MrKrasnov commented May 17, 2026 •

edited by veselcraft

Loading

MrKrasnov commented May 18, 2026 •

edited

Loading

MrKrasnov commented May 25, 2026 •

edited

Loading