Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Course import failing when plugin already present #37

Closed
joe-allen-89 opened this issue Jan 9, 2024 · 5 comments
Closed

Course import failing when plugin already present #37

joe-allen-89 opened this issue Jan 9, 2024 · 5 comments
Labels
bug An error has occurred high priority Should be prioritised over all other issues

Comments

@joe-allen-89
Copy link
Contributor

What happened?

When importing a course if any of the plugins being imported are already in the tool the import fails with the below error visible in the tool:

image

And the below errors in the console:

2024-01-09T12:31:51.080Z error adaptframework PLUGIN_IMPORT_FAILED C:\Working-AuthoringTool\240104\adapt-authoring\APP_DATA\temp\file-uploads\f3fa00c24854b0bd1cd945313_unzip\src\menu\adapt-contrib-boxMenu AdaptError: CONTENTPLUGIN_INSTALL_FAILED
at Object.get [as CONTENTPLUGIN_INSTALL_FAILED] (file:///C:/Working-AuthoringTool/240104/adapt-authoring/node_modules/adapt-authoring-errors/lib/ErrorsModule.js:50:20)
at ContentPluginModule.installPlugins (file:///C:/Working-AuthoringTool/240104/adapt-authoring/node_modules/adapt-authoring-contentplugin/lib/ContentPluginModule.js:246:29)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async file:///C:/Working-AuthoringTool/240104/adapt-authoring/node_modules/adapt-authoring-adaptframework/lib/AdaptFrameworkImport.js:463:34
at async Promise.all (index 0)
at async AdaptFrameworkImport.importCoursePlugins (file:///C:/Working-AuthoringTool/240104/adapt-authoring/node_modules/adapt-authoring-adaptframework/lib/AdaptFrameworkImport.js:453:7)
at async AdaptFrameworkImport.import (file:///C:/Working-AuthoringTool/240104/adapt-authoring/node_modules/adapt-authoring-adaptframework/lib/AdaptFrameworkImport.js:221:9)
at async importHandler (file:///C:/Working-AuthoringTool/240104/adapt-authoring/node_modules/adapt-authoring-adaptframework/lib/AdaptFrameworkUtils.js:219:24) {
code: 'CONTENTPLUGIN_INSTALL_FAILED',
statusCode: 500,
meta: {
description: 'Installation of plugins failed',
data: { errors: 'List of errors' }
},
data: { errors: [ [AdaptError] ] }
}

Expected behaviour

No response

Steps to reproduce

Import any course with a plugin already loaded into the tool

Authoring tool version

latest master

Framework version

5.33.13

What browsers are you seeing the problem on?

Firefox

Relevant log output

No response

@joe-allen-89 joe-allen-89 added the bug An error has occurred label Jan 9, 2024
@joe-allen-89 joe-allen-89 added the high priority Should be prioritised over all other issues label Jan 9, 2024
@taylortom taylortom moved this from New to Backlog in adapt-authoring: The TODO Board Jan 15, 2024
@taylortom taylortom self-assigned this Jan 29, 2024
@taylortom taylortom moved this from Backlog to Assigned in adapt-authoring: The TODO Board Jan 29, 2024
@taylortom
Copy link
Contributor

taylortom commented Feb 2, 2024

Can't replicate this locally, have tested with plugins with both the same and out-of-sync versions.

@joe-allen-89 Do you have any tips on how to replicate?

@taylortom taylortom moved this from Assigned to Awaiting Response in adapt-authoring: The TODO Board Feb 2, 2024
@chris-steele
Copy link
Contributor

chris-steele commented Apr 9, 2024

@joe-allen-89 are you running the tool on Windows? If so, we have identified a path issue that gives rise to this kind of error.

@taylortom taylortom removed their assignment Jun 19, 2024
@taylortom
Copy link
Contributor

Can this be closed?

@chris-steele
Copy link
Contributor

chris-steele commented Jun 19, 2024

looking at the OP error and trace, this is exactly what I had prior to the fix in #42

Propose this is closed @taylortom unless you can still replicate @joe-allen-89 ?

@taylortom
Copy link
Contributor

Cool, yep that's what I thought

@github-project-automation github-project-automation bot moved this from Awaiting Response to Merged in adapt-authoring: The TODO Board Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error has occurred high priority Should be prioritised over all other issues
Projects
Development

No branches or pull requests

3 participants