Sync resources - Some fields in 'Edit details' don't get updated after attempted sync

[pcenov]

Should be done in combination with: #4090

Observed behavior

A number of fields don’t get updated after having selected all available ‘sync’ options:
Language, Visible to, Provider, Aggregator, License description

Expected behavior

All fields should be updated as I've selected the option to update all file information.

Steps to reproduce the issue

1. Create a channel A and channel B.
2. Upload a file in channel A and then import the same file in channel B.
3. Edit all details of the file in channel A.
4. Go to channel B and select the option to sync all resources (select all checkboxes in the modal window).
5. Compare the details for both files.

Additional information

Usage Details

• OS: Windows 10
• Browser: Chrome
• URL: https://hotfixes.studio.learningequality.org/en/

[pcenov]

@bjester in case this issue was missed by everyone, as it has been left unassigned, just wanted to mention that it's still valid in the latest release

[bjester]

Related to this, we should actually prevent editing of the 'Source' fields, since a user shouldn't be able to change the licensing for a copied/imported resource.

We may still need to update the sync to write over these fields, to fix existing copied/imported issues, since they would become no longer editable with the referenced issue fix.

[akolson]

Hi @bjester! I am picking this up!

[bjester]

Thanks @akolson! I also assigned #4090 since it makes sense to do them together

[akolson]

Hi @bjester! Just confirming here before I open a pr that we should prevent edits for all "Source" fields when a resource has been copied/imported. A user should be able to edit the "Source" fields in other circumstances, is this correct?

[bjester]

@akolson Yes that sounds correct to me. Will you make these fields disabled?

[akolson]

@bjester disabled vs readonly?

[bjester]

@akolson Good thinking. readonly is more appropriate 👍

[bjester]

Closing because of #4189 and opened new remaining issue #4190