Project

General

Profile

Bug #103097

des CsvExportAfterJob sont exécutés mais ont toujours un statut "registered"

Added by Thomas Noël 9 days ago. Updated 2 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
07 March 2025
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Même situation que #103096 mais ici on a des jobs où on avait prévu de ne jamais les enregistrer ("avoid store of afterjob" en commentaire) :

        count = self.formdef.data_class().count()
        job = CsvExportAfterJob(
            self.formdef,
            fields=fields,
            selected_filter=selected_filter,
            selected_filter_operator=selected_filter_operator,
            user=user,
            query=query,
            criterias=criterias,
            order_by=order_by,
            skip_header_line=skip_header_line,
        )
        if count > self.WCS_SYNC_EXPORT_LIMIT:
            job = get_publisher().add_after_job(job)
            job.store()
            return redirect(job.get_processing_url())

        job.id = None  # avoid store of afterjob
        job.execute()
...

Or en examinant les afterjobs, on voit plein de CsvExportAfterJob en statut "registered", et ce sont qui ont moins de 100 lignes (self.WCS_SYNC_EXPORT_LIMIT).

Il semble que la technique « job.id = None » enregistre quand même l'afterjob. (Peut-être le passage en SQL ?)

Associated revisions

Revision a2340e73 (diff)
Added by Frédéric Péters 2 days ago

misc: use an explicit value to mark jobs not to be stored (#103097)

History

#1

Updated by Robot Gitea 8 days ago

  • Status changed from Nouveau to En cours
  • Assignee set to Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#2

Updated by Robot Gitea 8 days ago

  • Status changed from En cours to Solution proposée
#3

Updated by Robot Gitea 7 days ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 2 days ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#5

Updated by Transition automatique 2 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF