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

File names and links fixed #4

Closed
wants to merge 1 commit into from
Closed

Conversation

nikoloup
Copy link

Hey,

these are the changes I made overall so far. They are "fixing" the following issues we had with our blackboard export:

  • The filenames in moodle contained the blackboard resource_id (xid_@@@@@@_1), that didn't look very nice, so I am cleanng that up.
  • The links to the files were incorrect. They path and filename was a bit off and thus the file could not be accessed. I did more or less the same changes as above for cleaning up the filename and now they work fine.
  • In the quiz module, it is possible to embed images in questions. From what I figured, these links remained entirely unconverted and could not be loaded. To fix this, I replaced the blackboard link with the moodle path of the file directly in the .dat file. However, the course id in the moodle system is required to link to the file so as a workaround, I added an optional parameter that you can pass to reteach to send the course id in case the course has quizzes to be converted.

Unfortunately, I do not have access to our blackboard installation (just the exports) so I cannot really tell why it wasn't working from the start. Check out the changes when you have time and tell me what you think.

Thanks!!

@adamzap
Copy link
Owner

adamzap commented Apr 25, 2014

Thank you for sharing these changes.

I don't have access to Blackboard archives anymore either, so I can't test.

I have a feeling the issue was caused by your Blackboard or Moodle environment. I'll close this pull request and put a reference to it in the README so it may help others.

@adamzap adamzap closed this Apr 25, 2014
adamzap added a commit that referenced this pull request Apr 25, 2014
@nikoloup
Copy link
Author

That sounds excellent, thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants