Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-41603

Queuing multiple attachments to the content review service - errors prevent subsequent attachments from being queued

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 12.6, 19.2, 20.0
    • Fix Version/s: 12.7, 19.3, 20.0
    • Component/s: content review
    • Labels:
      None
    • 19 status:
      Resolved
    • 12 status:
      Resolved
    • Test Plan:
      Hide
      1. Create an assignment that uses TII and allows resubmissions
      2. Submit to this assignment using multiple attachments
      3. Resubmit to this assignment, adding a new attachment
      4. Ensure the new file has been queued with the content review service / is not in an error state
      Show
      Create an assignment that uses TII and allows resubmissions Submit to this assignment using multiple attachments Resubmit to this assignment, adding a new attachment Ensure the new file has been queued with the content review service / is not in an error state

      Description

      In ContentReviewQueueServiceImpl.queueContent(...), we throw a QueueException if an item already exists. If you're resubmitting multiple attachments, and some of these attachments already exist, this QueueException will abort the whole process. We should instead aggregate these and then just throw one exception after the valid items have been queued

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  bbailla2 Brian Baillargeon
                  Reporter:
                  bbailla2 Brian Baillargeon
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration