groove/Part-DB-server
1
0
Fork 0
mirror of https://github.com/Part-DB/Part-DB-server.git synced 2026-05-10 15:12:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Clean up stale pending jobs and add job ID to display

Browse source 
- Auto-delete pending jobs with 0 results (from failed searches/500 errors)
- Show job ID (#N) in manage page and step2 to distinguish identical jobs
- Move timestamp to subtitle line on manage page for cleaner layout
This commit is contained in:
Sebastian Almberg 2026-03-12 11:29:39 +01:00
parent 8e66f32ddc
commit 99148eee76
3 changed files with 6 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/Controller/BulkInfoProviderImportController.php
View file

@ -282,8 +282,8 @@ class BulkInfoProviderImportController extends AbstractController
$updatedJobs = true;
}
// Mark jobs with no results for deletion (failed searches)
if ($job->getResultCount() === 0 && $job->isInProgress()) {
// Mark jobs with no results for deletion (failed searches or stale pending)
if ($job->getResultCount() === 0 && ($job->isInProgress() || $job->isPending())) {
$jobsToDelete[] = $job;
}
}

3
templates/info_providers/bulk_import/manage.html.twig
View file

@ -97,7 +97,8 @@
{% set showCompletedAt = showCompletedAt|default(false) %}
<tr>
<td>
<strong>{{ job.displayNameKey|trans(job.displayNameParams) }} - {{ job.formattedTimestamp }}</strong>
<strong>#{{ job.id }} - {{ job.displayNameKey|trans(job.displayNameParams) }}</strong>
<br><small class="text-muted">{{ job.formattedTimestamp }}</small>
</td>
<td>{{ job.partCount }}</td>
<td>{{ job.resultCount }}</td>

4
templates/info_providers/bulk_import/step2.html.twig
View file

@ -9,7 +9,7 @@
{% block card_title %}
<i class="fas fa-search"></i> {% trans %}info_providers.bulk_import.step2.title{% endtrans %}
<span class="badge bg-secondary">{{ job.displayNameKey|trans(job.displayNameParams) }} - {{ job.formattedTimestamp }}</span>
<span class="badge bg-secondary">#{{ job.id }} - {{ job.displayNameKey|trans(job.displayNameParams) }}</span>
{% endblock %}
{% block card_content %}
@ -44,7 +44,7 @@
}) }}>
<div class="d-flex justify-content-between align-items-center mb-3">
<div>
<h5 class="mb-1">{{ job.displayNameKey|trans(job.displayNameParams) }} - {{ job.formattedTimestamp }}</h5>
<h5 class="mb-1">#{{ job.id }} - {{ job.displayNameKey|trans(job.displayNameParams) }}</h5>
<small class="text-muted">
{{ job.partCount }} {% trans %}info_providers.bulk_import.parts{% endtrans %} •
{{ job.resultCount }} {% trans %}info_providers.bulk_import.results{% endtrans %} •