Create comand #1799

GreedVal · 2025-11-06T17:00:14Z

Data export from the database for analytics has been implemented via the admin panel. Data is downloaded as a CSV file.

fey · 2025-11-17T15:45:25Z

@GreedVal привет, как успехи, нужна помощь?

GreedVal · 2025-11-17T16:03:25Z

@GreedVal привет, как успехи, нужна помощь?

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

fey · 2025-12-09T09:30:07Z

@GreedVal ЭЭэххх(((( Грустненько. А тут по задачке в целом получилось продвинуться, или все в зародыше? Можешь оставить пометки-заметки в задача-ишшусе, чтобы кто-то следующий мог взять в работу?

GreedVal · 2025-12-09T09:34:09Z

@GreedVal ЭЭэххх(((( Грустненько. А тут по задачке в целом получилось продвинуться, или все в зародыше? Можешь оставить пометки-заметки в задача-ишшусе, чтобы кто-то следующий мог взять в работу?

@fey Я вроде разгреб, долги на работе вчера сделал пул риквест по багу в регистрации, сегодня займусь выгрузкой, так что вроде в строю)

GreedVal · 2025-12-10T18:21:47Z

Сделал сервис для выгрузки таблиц в csv файлы, запускается через artisan command

fey · 2025-12-11T11:42:41Z

@GreedVal можешь показать пример того, что выгрузится?
Плюс тесты упали - хщз в чем дело, можно обновить ветку (через ребейз и форс пуш).

Еще тестов не хватает.
Еще обрати внимание - экспорт на диск может не сработать - на хероку нет файловой системы как таковой.
Надо подумать, как данные можно выгрузить (можно на почту отправлять).

GreedVal · 2025-12-11T16:46:10Z

пример выгрузки

id,name,email,github_name,hexlet_nickname,points,created_at
"1","test","[email protected]","","","0","2025-09-27 18:24:19"
"2","greedval","[email protected]","","","0","2025-09-29 18:08:38"

я думал вообще сделать выгрузку через админку кнопкой, про тесты забыл сделаю

GreedVal · 2025-12-15T19:14:44Z

Сделал выгрузку данных для аналитики, через админ панель. выгрузка данных через класс сервис AnalyticsExporter, его использует контролер.

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

Тесты для контролера и сервиса написаны отдельно.

На рендер не получается залить ругается на postgres почемуто еще не разобрался.

fey · 2025-12-16T18:14:44Z

app/Http/Controllers/Admin/ExportController.php

+            'type' => 'required|string',
+        ]);
+
+        $directory = 'exports/' . now()->format('Y-m-d_H-i-s');


формирование пути к директории экспорта получается разделилось на несколько частей. используй массив + склейку слешем для создания пути к директории

fey · 2025-12-16T18:16:21Z

app/Services/AnalyticsExporter.php

+
+        foreach ($collection as $item) {
+            $quoted = array_map(
+                static fn($value): string => '"' . str_replace('"', '""', (string) $value) . '"',


тут стоит поправить, (здесь и далее). Вообще лучше использовать готовое решение для работы с CSV

fey · 2025-12-16T18:17:45Z

database/factories/ChapterFactory.php

+use App\Models\Chapter;
+use Illuminate\Database\Eloquent\Factories\Factory;
+
+class ChapterFactory extends Factory


Хм, а у нас ведь нет особо необходимости в генерации глав и упражнений. Они ведь есть в коде. Они не подходят?

fey · 2025-12-16T18:18:52Z

routes/web.php

        Route::resource('solutions', 'SolutionController')->only('index');
+
+        Route::get('export', 'ExportController@index')->name('export.index');
+        Route::post('export', 'ExportController@export')->name('export');


давай использовать ресурсный роутинг.

fey · 2025-12-16T18:25:37Z

app/Services/AnalyticsExporter.php

+        Storage::makeDirectory($directory);
+
+        switch ($type) {
+            case 'users':


здесь мы выбираем не тип, а имя таблицы, которую будем экспортировать, или имя сущности. плюсом вместо строки- лучше использовать enum/константы или что-то такое. Например даже имя класса подойдет.

fey · 2025-12-16T18:28:01Z

tests/Feature/Http/Controllers/Admin/ExportControllerTest.php

+        $this->assertStringContainsString((string)$user->id, $content);
+
+        unlink($filePath);
+        rmdir($latestFolder);


вот это можно вынести в tear down метод (очистку)

fey · 2025-12-16T18:29:46Z

tests/Feature/Http/Controllers/Admin/ExportControllerTest.php

+        rsort($folders);
+        $latestFolder = $folders[0];
+
+        $filePath = $latestFolder . '/users.csv';


снова конкатенация.

С путями самый удобный и простой способ - этомассив + join, или интерполяция.

Кстати тут советую подумать над тем, как можно не искать директорию, как ты делаешь. Например в экспортер можно передавать директорию, куда он будет сохранять, можно замораживать время в тестах (тк у тебя директория зависит от текущего времени)

fey · 2025-12-16T18:31:08Z

app/Services/AnalyticsExporter.php

+                $this->exportComments("{$directory}/comments.csv");
+                break;
+            case 'activity':
+                $this->exportActivityLog("{$directory}/activity.csv");


как будто бы можно не передавать каждый раз директорию, а записать ее в свойство класса, и брать ее при сохранении. Т.е. у тебя экспортер будет иметь настройку, куда он может экспортировать данные.

ну и можно тут обобщить, сделать массив, где ключ - это то что щас в кейсе, и использовать анонимные функции для экспорта. ну, такой продвинутый вариант.

fey · 2025-12-16T18:33:50Z

app/Http/Controllers/Admin/ExportController.php

+
+        $this->exporter->export($directory, $request->type);
+
+        $fileName = "{$request->type}.csv";


несклько обращений к request->type - можно выделить в переменную.

fey · 2025-12-19T11:35:16Z

app/Http/Controllers/Admin/ExportController.php


        $fileName = "{$request->type}.csv";
-        $filePath = storage_path("app/{$directory}/{$fileName}");
+        $filePath = storage_path(implode('/', ['app', $directory, $fileName]));


Давай тут в методе нейминг поправим, потому что, щас у тебя есть некая directory - и непонятно, за что она отвечает, потому что в нее проихсодит экспорт на 38 строке, а потом внезапно на 41 создается новый путь, но уже storage/app - выглядит запутано.

Да, я уже думая что у меня совсем логика запутана так как и в сервисе и в контролере отдельно создается путь к сохраненному файлу из за чего неразбериха, наверное правильно будет чтобы сервис возвращал путь к фалу а в контролере я просто его использовал. Так меньше шансов не правильного поведения контролера.

Так в контролер вообще не будет участвовать в формировании пути

По фабрикам ChapterFactory.php создавал для тестов но сейчас думаю а зачем аналитикам вообще их выгружать, возможно стоит убрать их в export, а потом если нужно будет добавить

Now the path is created in the service and the controller accepts the already prepared path to the file

Сделаны тесты под новую логику export

fey · 2025-12-22T10:01:32Z

app/Services/AnalyticsExporter.php

+        );
+
+        $csv->setDelimiter(',');
+        $csv->setNewline("\n");


тут все еще самописная работа с csv. Давай возьмем готовое решение, вот к примеру. https://github.com/thephpleague/csv

fey · 2025-12-22T10:02:48Z

tests/Feature/Services/AnalyticsExporterTest.php

+            'activity'  => 'export/activity.csv',
+        ];
+
+        foreach ($types as $type => $path) {


тут подойддет data provider

fey · 2025-12-22T10:04:19Z

tests/Feature/Services/AnalyticsExporterTest.php

+        }
+    }
+
+    public function testUsersCsvContainsCorrectHeadersAndRow(): void


не думмаю, что имеет смысл отедльно тестировать только для юзеров это.

Смотри, у тебя ведь общая логика для экспорта одна, везде создаются заголовки.
Просто создай фикстуру, и с ней сравнивай результат экспорта. Ну и дата провайдер тебе в этом плане поломожет организовать тест.

fey · 2025-12-22T10:06:01Z

@GreedVal осмотри по тестам как там работа с activity происходит. Они вроде как генерируются через што-то, чтобы данные консистентные были, т.е. если у юзер должен оставить коммент (перед тестами), то и есть нужная "активность".
Тут можно без проблем сиды использовать, без лишних фабрик. (они могут пригождаться, например для генерации данных).

AnalyticsExporterTest made via seed, activity factory removed

Create comand

0d45ae9

Merge branch 'main' into dump-database

377cb7a

GreedVal added 5 commits December 9, 2025 18:25

Merge branch 'main' into dump-database

20bf344

Create class services

e74bd64

fix spase

d2127a0

Fix select

92417f8

Merge branch 'main' into dump-database

c438721

GreedVal marked this pull request as ready for review December 10, 2025 17:42

Make test

2edff8e

GreedVal added 8 commits December 11, 2025 22:08

Fix test create factory

031f705

Updated dependencies

98ed224

Fix factory

2894ccf

fix export services

f7fd033

Fix test

713fb1e

Fix test

1b03f18

Merge branch 'main' into dump-database

1d1e580

Merge branch 'main' into dump-database

8d79468

GreedVal marked this pull request as draft December 14, 2025 13:38

GreedVal added 5 commits December 14, 2025 19:32

fix compos

5c4cbb4

fix

176abf4

Make export in admin panel

6fb40f7

Merge branch 'main' into dump-database

86a8a7b

Fix name in test csv file

ffbe573

GreedVal added 2 commits December 15, 2025 21:41

fix

3ce546f

fix

a00a2b2

GreedVal marked this pull request as ready for review December 15, 2025 19:15

fey reviewed Dec 16, 2025

View reviewed changes

GreedVal marked this pull request as draft December 17, 2025 07:38

GreedVal added 2 commits December 18, 2025 19:44

path formation fix

2d42092

Creating a CSV file using a library

bafb7a7

fey reviewed Dec 19, 2025

View reviewed changes

GreedVal added 3 commits December 19, 2025 23:57

Routes used resource have been corrected

10da35d

The controller and service have been replaced.

f9f7710

Now the path is created in the service and the controller accepts the already prepared path to the file

Fix test

5931845

Сделаны тесты под новую логику export

fey reviewed Dec 22, 2025

View reviewed changes

Fix test

c3ca73f

AnalyticsExporterTest made via seed, activity factory removed


		$this->exporter->export($directory, $request->type);

		$fileName = "{$request->type}.csv";

Create comand #1799

Are you sure you want to change the base?

Create comand #1799

Uh oh!

Conversation

GreedVal commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

fey commented Nov 17, 2025

Uh oh!

GreedVal commented Nov 17, 2025

Uh oh!

fey commented Dec 9, 2025

Uh oh!

GreedVal commented Dec 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

GreedVal commented Dec 10, 2025

Uh oh!

fey commented Dec 11, 2025

Uh oh!

GreedVal commented Dec 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

GreedVal commented Dec 15, 2025

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!

fey Dec 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

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!

fey commented Dec 22, 2025

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

2 participants

GreedVal commented Nov 6, 2025 •

edited

Loading

GreedVal commented Dec 9, 2025 •

edited

Loading

GreedVal commented Dec 11, 2025 •

edited

Loading

fey Dec 16, 2025 •

edited

Loading