IMAGEDAM-1982: Prevent the resend of capture images and restrict icons

[Conalb97]

What does this change?

This change prevents users from sending images that were received from Photo Sales back to Photo Sales - this is enabled through the same modal/notification mechanism that is already applied to images that have an 'Added to Photo Sales' usage. Additionally, the 'sales' icon (for BBC Photo Sales) will now also display for images received from Photo Sales, whereas before it was only present for images that were sent to Photo Sales. Finally it ensures that only archivist users will be able to see the sales icon, standard users will not be able to.

How should a reviewer test this change?

As a user with elevated permissions and with the showSendToPhotoSales flag set to true:

Users can not send images back to BBC Photo Sales:

• Apply the 'uploader' chip and enter 'Capture_AutoIngest'
• The query will be executed and the UI will refresh
• Select an image from the returned images
• Select the 'Send to Photo Sales' button
• A notification banner appears informing the user that the image cannot be sent as it already exists in BBC Photo Sales

The sales icon is present for images received from Photo Sales:

• Apply the 'uploader' chip and enter 'Capture_AutoIngest'
• The query will be executed and the UI will refresh
• Confirm that the sales icon is present on the returned images

As a user without elevated permissions and with the showSendToPhotoSales flag set to true:

The sales icon is not present on any images:

• Apply the 'uploader' chip and enter 'Capture_AutoIngest'
• The query will be executed and the UI will refresh
• Confirm that the sales icon is not present on any images
• Remove the chip
• Apply the 'usage@platform' chip and select 'Added to Photo Sales'
• The query will be executed and the UI will refresh
• Confirm that the sales icon is not present on any images

Who should look at this?

Tested? Documented?

• [ ] locally by committer
• [ ] locally by Guardian reviewer
• [ ] on the Guardian's TEST environment
• [ ] relevant documentation added or amended (if needed)

[AndyKilmory]

Looks good to me - but given the increased usage of the media API call to understand user permissions we must make sure that this value is cached at the session level and we don't repeatedly make calls to the server to get these data

[prout-bot]

Seen on auth, usage, image-loader, metadata-editor, thrall, leases, cropper, collections, media-api, kahuna (created by @Conalb97 and merged by @andrew-nowak 8 minutes and 34 seconds ago) Please check your changes!