Comment reproduire :
Que se passe-t-il : "Le champ du numéro de lot est vide"
Ce qui est attendu : Le numéro de lot doit être « 12345 ».
Je peux confirmer ce problème.
Je viens de créer un dump HAR de firefox pour que cela soit documenté. Pendant massRemoveStock
le numéro de lot semble ne pas être transféré, si je le vois correctement. Il doit donc être implémenté dans le déroulement du projet (pour être présent lors du clic sur "retirer les pièces du stock".
Bonjour! J'ai vu ce problème sur bountysource alors j'ai jeté un œil au code.
J'ai pu reproduire le bug. En inspectant http://partkeepr.local/api/project_run_parts/1
j'ai vu que lotNumber
est défini dans la partie à l'intérieur du projet, mais pas dans la partie de niveau supérieur. Sur cette base, je pense que c'est juste un problème d'affichage plutôt que la disparition de la valeur lotNumber
, en effet lotNumber
est copié à l'intérieur de massRemoveStockAction
de src/PartKeepr/PartBundle/Controller/PartController.php
.
À quoi ressemble le correctif suivant ? https://github.com/partkeepr/PartKeepr/pull/1153
Ce serait bien d'avoir la confirmation que votre RP résout complètement ce problème !
J'ai essayé de mettre en œuvre les changements mais cela n'a pas semblé avoir d'effet. J'ai seulement implémenté les modifications et fait une réexécution de la page /setup. Dois-je également exécuter des commandes de composition ?
Je pense que le cache des fichiers javascript frontend doit être supprimé pour que cette modification s'applique. Je viens de les supprimer puis de relancer la configuration, mais plus tard, j'ai trouvé ces commandes qui pourraient mieux fonctionner https://wiki.partkeepr.org/wiki/Running_PartKeepr_from_GIT#Console_commands
@ed-commits Je suis un noob total avec cet environnement, donc désolé pour mes questions stupides. Mais en supprimant le cache javascript frontal, voulez-vous dire exécuter la commande rm -rf app/cache/*
? J'ai essayé ceci avant d'exécuter la configuration, mais je ne parviens toujours pas à inclure le numéro de lot dans les exécutions du projet. Pouvez-vous me fournir la procédure exacte que vous avez utilisée pour vérifier le patch afin que je puisse le reproduire ?
Je l'ai fait, aussi peut-être essayer rm -rf web/js/compiled
et rm -rf web/js/packages/extjs6
aussi. puis relancez l'installation. alors le changement devrait s'appliquer.
@ed-commits J'ai maintenant essayé cela aussi, et je ne suis malheureusement pas en mesure de voir que le changement fait une différence. Quelqu'un d'autre peut-il confirmer que cela résout le problème ?
Je peux maintenant confirmer que #1153 résout ce problème. On dirait que le cache quelque part a été réinitialisé après un certain temps. Cependant, supprimer compilé et extjs6 n'est pas recommandé. Cela bloquera l'ensemble du système, le rendant bloqué dans la page de chargement, et la page de configuration est restée vide. Ce problème peut être fermé une fois le correctif fusionné avec le maître.
edit: Obs, je vois maintenant que le numéro de lot dans le projet Run changera si le projet change (sous edit-> projects). Ce n'est pas correct. Le numéro de lot ne doit pas pouvoir être modifié après l'exécution d'un projet. Donc pas directement lié au projet, si cela a du sens.
Je pense que le problème est en effet quelque part dans l'action de retrait en masse
ici : https://github.com/partkeepr/PartKeepr/blob/e39c5f87f9ad44c7b7d4ffb521178f492761320d/src/PartKeepr/PartBundle/Controller/PartController.php#L95
Ou plutôt, que le champ $removal->lotNumber
est vide. Donc plus précisément quelque part dans la requête json
https://github.com/partkeepr/PartKeepr/blob/e39c5f87f9ad44c7b7d4ffb521178f492761320d/src/PartKeepr/PartBundle/Controller/PartController.php#L33
Ce qui, je suppose, est défini quelque part ici :
https://github.com/partkeepr/PartKeepr/blob/7dd3ef8f2395097b3659bbe0587eac70b6ff7671/src/PartKeepr/FrontendBundle/Resources/public/js/Components/Project/ProjectReportResultGrid.js#L357
Juste au cas où cela serait utile,
Je remarque qu'il y a une sorte de problème de logique avec le bouton "Enregistrer le projet" lors de l'importation d'une pièce dans le projet.
Une fois tout le processus effectué (lecture du fichier CSV, cliquez sur "Exécuter l'import") réussi pour ajouter la liste des pièces au projet, la table ProjectPart est correctement remplie, même la fermeture de la fenêtre d'import reste correcte mais lorsque vous cliquez sur le " Bouton Enregistrer le projet" pour décharger/annuler les modifications (?)
à la fin de cette page est expliqué comment les utilisateurs contournent ce
https://readthedocs.web.cern.ch/display/PARTK/07a+-+Creating+Projects+and+BOM+Imports
la mention du "save project" au début de ce numéro et le comportement décrit semblent similaires à ce que j'ai décrit.
Salutations
Cher JoarGjersund et ed-commit
Si j'ai bien compris lorsque vous exécutez http://localhost/web/app_dev.php
https://readthedocs.web.cern.ch/display/PARTK/Setup+for+Debug+and+Verbose+mode
vous ne vous souciez pas du cache, ce que vous exécutez se fait directement.
Salutations
Commentaire le plus utile
Bonjour! J'ai vu ce problème sur bountysource alors j'ai jeté un œil au code.
J'ai pu reproduire le bug. En inspectant
http://partkeepr.local/api/project_run_parts/1
j'ai vu quelotNumber
est défini dans la partie à l'intérieur du projet, mais pas dans la partie de niveau supérieur. Sur cette base, je pense que c'est juste un problème d'affichage plutôt que la disparition de la valeurlotNumber
, en effetlotNumber
est copié à l'intérieur demassRemoveStockAction
desrc/PartKeepr/PartBundle/Controller/PartController.php
.À quoi ressemble le correctif suivant ? https://github.com/partkeepr/PartKeepr/pull/1153