Skip to content

Commit 364b889

Browse files
codedmonkeycodedmonkey
committed
Merge branch 'packages-mirroring-form'
2 parents 607fa97 + c0f0019 commit 364b889

6 files changed

Lines changed: 174 additions & 28 deletions

File tree

‎assets/controllers/packages_mirroring_controller.ts‎

Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
import { Controller } from '@hotwired/stimulus';
2+
3+
/* stimulusFetch: 'lazy' */
4+
export default class extends Controller {
5+
static targets = ['form', 'packageList'];
6+
static values = {
7+
packageUrl: String,
8+
};
9+
10+
formTarget: HTMLFormElement;
11+
packageListTarget: HTMLElement;
12+
packageUrlValue: string;
13+
14+
submitForm(event: Event) {
15+
event.preventDefault();
16+
17+
const formData = new FormData(this.formTarget);
18+
19+
const data = [];
20+
for (const [key, value] of formData) {
21+
data.push(`${encodeURIComponent(key)}=${encodeURIComponent(value.toString())}`);
22+
}
23+
const body = data.join('&');
24+
25+
fetch(this.formTarget.getAttribute('action'), {
26+
method: 'POST',
27+
headers: {'Content-Type': 'application/x-www-form-urlencoded'},
28+
body,
29+
})
30+
.then(response => response.json())
31+
.then((data: PackageResultResponse) => {
32+
for (const result of data.results) {
33+
const packageUrl = this.packageUrlValue.replace('place/holder', result.packageName);
34+
35+
const packageName = !result.error ? `<a href="${packageUrl}" target="_blank">${result.packageName}</a>` : result.packageName;
36+
const registryName = result.registryName || '';
37+
let message = result.message;
38+
if (result.error) {
39+
message = `<span class="text-warning">${message}</span>`;
40+
} else if (result.created) {
41+
message = `<span class="text-success">${message}</span>`;
42+
}
43+
44+
const listItem = document.createElement('div');
45+
listItem.innerHTML = `
46+
<div class="row my-2">
47+
<div class="col-md-3 col-lg-2">${packageName}</div>
48+
<div class="col-md-3 col-lg-2">${registryName}</div>
49+
<div class="col-md-6 col-lg-8">${message}</div>
50+
</div>
51+
`;
52+
53+
this.packageListTarget.appendChild(listItem);
54+
}
55+
});
56+
57+
const packagesInput: HTMLTextAreaElement = this.formTarget.querySelector('[name="package_add_mirroring_form[packages]"]');
58+
packagesInput.value = '';
59+
}
60+
}
61+
62+
interface PackageResultResponse {
63+
results: PackageResult[];
64+
}
65+
66+
interface PackageResult {
67+
packageName: string;
68+
registryName: string|null;
69+
message: string;
70+
created: boolean;
71+
error: boolean;
72+
}

‎src/Controller/Dashboard/DashboardPackagesController.php‎

Lines changed: 29 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ public function statistics(string $packageName): Response
140140

141141
#[Route('/dashboard/packages/add-mirroring', name: 'dashboard_packages_add_mirroring')]
142142
#[IsGranted('ROLE_ADMIN')]
143-
public function addMirror(Request $request): Response
143+
public function addMirroring(Request $request): Response
144144
{
145145
$form = $this->createForm(PackageAddMirroringFormType::class);
146146

@@ -149,25 +149,43 @@ public function addMirror(Request $request): Response
149149
if ($form->isSubmitted() && $form->isValid()) {
150150
$registry = $form->get('registry')->getData();
151151

152-
$packageNames = explode(PHP_EOL, $form->get('packages')->getData());
153-
$packageNames = array_map('trim', $packageNames);
152+
$packageNamesInput = $form->get('packages')->getData();
153+
$packageNames = preg_split('#(\s|,)+#', $packageNamesInput);
154154

155155
$results = [];
156156

157157
foreach ($packageNames as $packageName) {
158-
if (null !== $this->packageRepository->findOneBy(['name' => $packageName])) {
158+
if (!preg_match('#[a-z0-9_.-]+/[a-z0-9_.-]+#', $packageName)) {
159159
$results[] = [
160+
'packageName' => $packageName,
161+
'registryName' => null,
162+
'created' => false,
160163
'error' => true,
161-
'message' => "The package $packageName already exists and was skipped",
164+
'message' => "The package name $packageName is invalid.",
165+
];
166+
167+
continue;
168+
}
169+
170+
if (null !== $this->packageRepository->findOneBy(['name' => $packageName])) {
171+
$results[] = [
172+
'packageName' => $packageName,
173+
'registryName' => null,
174+
'created' => false,
175+
'error' => false,
176+
'message' => "The package $packageName already exists and was skipped.",
162177
];
163178

164179
continue;
165180
}
166181

167182
if (!$this->metadataResolver->provides($packageName, $registry)) {
168183
$results[] = [
184+
'packageName' => $packageName,
185+
'registryName' => $registry->getName(),
186+
'created' => false,
169187
'error' => true,
170-
'message' => "The package $packageName could not be found and was skipped",
188+
'message' => "The package $packageName could not be found and was skipped.",
171189
];
172190

173191
continue;
@@ -183,14 +201,17 @@ public function addMirror(Request $request): Response
183201
$this->messenger->dispatch(new UpdatePackage($package->getId()));
184202

185203
$results[] = [
204+
'packageName' => $packageName,
205+
'registryName' => $registry->getName(),
206+
'created' => true,
186207
'error' => false,
187-
'message' => "The package $packageName was created successfully",
208+
'message' => "The package $packageName was created successfully.",
188209
];
189210

190211
$this->entityManager->flush();
191212
}
192213

193-
return $this->render('dashboard/packages/add_mirroring_results.html.twig', [
214+
return $this->json([
194215
'results' => $results,
195216
]);
196217
}

‎templates/dashboard/dashboard_form_theme.html.twig‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
{% extends 'bootstrap_5_layout.html.twig' %}
2+
3+
{%- block form_errors -%}
4+
{%- if errors|length > 0 -%}
5+
{%- for error in errors -%}
6+
<div class="{% if form is not rootform %}invalid-feedback{% else %}invalid-feedback mb-3{% endif %} d-block">
7+
<span class="fa-regular fa-circle-xmark me-1" aria-hidden="true"></span>
8+
{{ error.message }}
9+
</div>
10+
{%- endfor -%}
11+
{%- endif -%}
12+
{%- endblock form_errors -%}

‎templates/dashboard/packages/add_mirroring.html.twig‎

Lines changed: 28 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,36 @@
11
{% extends '@EasyAdmin/page/content.html.twig' %}
22

3+
{% form_theme form 'dashboard/dashboard_form_theme.html.twig' %}
4+
35
{% block page_title %}Mirror packages from registry{% endblock %}
46

57
{% block page_content %}
6-
{{ form_start(form) }}
7-
{{ form_row(form.packages, {attr: {rows: 10}}) }}
8+
<div {{ stimulus_controller('packages-mirroring', {packageUrl: dashboard_path('dashboard_packages_info', {packageName: 'place/holder'})}) }}>
9+
<div
10+
class="border-top border-bottom my-3"
11+
{{ stimulus_target('packages-mirroring', 'packageList') }}
12+
>
13+
</div>
14+
15+
{{ form_start(form, {
16+
attr: {
17+
'action': dashboard_path('dashboard_packages_add_mirroring'),
18+
'data-action': 'packages-mirroring#submitForm',
19+
'data-packages-mirroring-target': 'form',
20+
},
21+
}) }}
22+
{{ form_errors(form) }}
823

9-
{{ form_row(form.registry) }}
24+
<div class="row">
25+
<div class="col-lg-8">
26+
{{ form_row(form.packages, {attr: {rows: 5}}) }}
27+
</div>
28+
<div class="col-lg-4">
29+
{{ form_row(form.registry) }}
30+
</div>
31+
</div>
1032

11-
<button class="btn btn-primary">Add packages</button>
12-
{{ form_end(form) }}
33+
<button class="btn btn-primary">Add packages</button>
34+
{{ form_end(form) }}
35+
</div>
1336
{% endblock %}

‎templates/dashboard/packages/add_mirroring_results.html.twig‎

Lines changed: 0 additions & 15 deletions
This file was deleted.

‎tests/FunctionalTests/DashboardPackagesControllerTest.php‎

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace CodedMonkey\Dirigent\Tests\FunctionalTests;
44

55
use CodedMonkey\Dirigent\Doctrine\Repository\PackageRepository;
6+
use CodedMonkey\Dirigent\Doctrine\Repository\RegistryRepository;
67
use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
78

89
class DashboardPackagesControllerTest extends WebTestCase
@@ -29,6 +30,38 @@ public function testStatistics(): void
2930
$this->assertAnySelectorTextSame('#total_today .display-6', number_format($package->getInstallations()->getData()[$todayKey] ?? 0, thousands_separator: ' '));
3031
}
3132

33+
public function testAddMirroring(): void
34+
{
35+
$client = static::createClient();
36+
$this->loginUser('admin');
37+
38+
$registry = $client->getContainer()->get(RegistryRepository::class)->findOneBy(['name' => 'Packagist']);
39+
40+
$client->request('GET', '/?routeName=dashboard_packages_add_mirroring');
41+
$client->submitForm('Add packages', [
42+
'package_add_mirroring_form[packages]' => 'psr/cache',
43+
'package_add_mirroring_form[registry]' => $registry->getId(),
44+
]);
45+
46+
$this->assertResponseStatusCodeSame(200);
47+
48+
// todo the submit request should be invoked with ajax, and this assertion should be performed on the initial request
49+
// however, the assertion is performed on the ajax response making it invalid
50+
// $this->assertAnySelectorTextSame(
51+
// '.text-success',
52+
// 'The package psr/cache was created successfully.',
53+
// 'A message showing the package was created must be shown.',
54+
// );
55+
56+
/** @var PackageRepository $packageRepository */
57+
$packageRepository = $client->getContainer()->get(PackageRepository::class);
58+
59+
$package = $packageRepository->findOneByName('psr/cache');
60+
self::assertNotNull($package, 'A package was created.');
61+
62+
$packageRepository->remove($package, true);
63+
}
64+
3265
public function testAddVcsRepository(): void
3366
{
3467
$client = static::createClient();

0 commit comments

Comments
 (0)