Extraneous strings on the Edit details page when editing HTML5 apps

[radinamatic]

Observed behavior

For the HTML5 apps there's an extraneous (or I'm forgetting what it was supposed to be) link with ConstantStrings.imscp_zip string 🤔

Errors and logs

…

Expected behavior

There should be no extraneous strings.

User-facing consequences

Incomplete resource file information

Steps to reproduce

…

Usage Details

• OS: Windows
• Browser: Firefox & Chrome
• tested with Kolibri QA channel (nakav-mafak) and other manually curated channels

@marcellamaki suggested that it might be related to #4205

[akolson]

@radinamatic It looks like the ConstantStrings.imscp_zip doesn't exist in

studio/contentcuration/contentcuration/frontend/shared/mixins.js

Line 89 in 69be8e5

export const constantStrings = createTranslator('ConstantStrings', {

.

For the HTML5 apps there's also an extraneous (or I'm forgetting what it was supposed to be) link with ConstantStrings.imscp_zip string

It would be good to establish what its intended purpose was before we remove it although I suspect this is just another file that was uploaded. In that case adding the String would suffice. Anyone recall anything? cc @rtibbles @bjester

[akolson]

Also @radinamatic, I am not able to replicate the "Unknown filename" link issue. I seem to consistently get the file name

[radinamatic]

Both @pcenov and I are currently replicating the "Unknown filename" link in Kolibri QA channel in ASB books and Flexbooks at least (Firefox and Chrome)

[akolson]

Thanks replicated.

[akolson]

It turns out that the these particular resources have "file" as their name. Any resource with "file" is defaulted to Unknown filename, which is a little misleading, maybe setting the string to something else would make more sense to the user. Happy to hear thoughts here.

studio/contentcuration/contentcuration/frontend/channelEdit/views/files/FileUploadItem.vue

Lines 137 to 143 in b9f89e9

	formattedFileDisplay() {
	const fileName = this.fileDisplay.original_filename;
	if (fileName === 'file' || !fileName) {
	return this.$tr('unknownFile');
	}
	return fileName;
	},

[bjester]

I don't think Unknown filename is a bug, in regards to what was fixed previously #3602 because the prior reported issue was that this was untranslated. We simply don't have any more information to provide to the user.

I think addressing the extraneous option for HTML5 apps is higher priority.

[akolson]

I have update and split the issue so they can be handled separately.