[DONE] Fix migration script for bbb #1214
Conversation
There was a problem hiding this comment.
Bonjour, merci pour ces modifications. Dans notre université, la base de Moodle est Postgre, je ne peux alors tester, mais cela me semble bon.
There was a problem hiding this comment.
Chez moi c'est l'inverse : ma base est en mysql et je n'ai rien pour tester postgre :)
There was a problem hiding this comment.
Cela fonctionne avec cette modification dans tous les cas ?
Typiquement, ce code est prévu pour gérer les anciens enregistrements du type :
https://scalelite.univ.fr/playback/presentation/2.0/playback.html?meetingId=XXXX
et devrait les convertir en :
https://scalelite.univ.fr/playback/presentation/2.3/XXXX
En regardant de plus près, il ne faudrait pas mettre à minima ce code :
source_video_url = source_url.replace(
"presentation/2.0/playback.html?meetingId=", "presentation/2.3/"
)
???
Merci
There was a problem hiding this comment.
Alors j'avoue que je n'ai pas compris l'origine de cette réécriture du lien. Quand je veux importer la présentation dans Pod (à partir de notre BBB 2.2), avec la réécriture, bbb-recorder me renvoie l'erreur
Error: Evaluation failed: TypeError: Cannot read properties of null (reading 'duration')
at __puppeteer_evaluation_script__:2:72
at ExecutionContext._evaluateInternal (/home/pod/bbb-recorder/node_modules/puppeteer/lib/ExecutionContext.js:122:13)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async ExecutionContext.evaluate (/home/pod/bbb-recorder/node_modules/puppeteer/lib/ExecutionContext.js:48:12)
at async main (/home/pod/bbb-recorder/export.js:142:27)
-- ASYNC --
at ExecutionContext.<anonymous> (/home/pod/bbb-recorder/node_modules/puppeteer/lib/helper.js:111:15)
at DOMWorld.evaluate (/home/pod/bbb-recorder/node_modules/puppeteer/lib/DOMWorld.js:112:20)
-- ASYNC --
at Frame.<anonymous> (/home/pod/bbb-recorder/node_modules/puppeteer/lib/helper.js:111:15)
at Page.evaluate (/home/pod/bbb-recorder/node_modules/puppeteer/lib/Page.js:860:43)
at Page.<anonymous> (/home/pod/bbb-recorder/node_modules/puppeteer/lib/helper.js:112:23)
at main (/home/pod/bbb-recorder/export.js:142:38)
Par contre en laissant le lien original, ça marche sans problème.
Donc pour bbb-recorder, le lien à utiliser pour les BBB < 2.3 me semble bien https://scalelite.univ.fr/playback/presentation/2.0/playback.html?meetingId=XXXX
Si besoin je peux te fournir le lien d'une présentation de notre BBB en 2.2 pour tester
There was a problem hiding this comment.
Si cela ne dérange pas, oui, je veux bien tester avec ton lien; j'ai aussi un lien d'un vieux test que je peux t'envoyer.
Sinon, cette erreur de duration me parle; cela venait d'une temporisation insuffisante (cf. https://www.esup-portail.org/wiki/pages/viewpage.action?pageId=914423810, premier paragraphe).
## New functionalities: * Use Whisper to format vtt subtitle (#1187) * Remove BBB module and add sipmediagw feature (#1190) * Add NOTIFY_SENDER boolean parameter (#1192) * Add fields to recording rest response (#1193) * Add return thumbnail url,width and height to oembed (#1194) * Add optional proxy URL for request coming from Aristote (#1218) ## Bugs corrected: * Change all filter_fields = () in filterset_fields = [], as filter_fields is deprecated (#1191) * Fix multi carousel (#1200) * Change regroup videos by theme when ORGANIZE_BY_THEME = True (#1203) * Bug habillage (#1211) * Fix migration script for BBB (#1214) * Improve RSS Feeds (#1215) * Fix BBB meeting deletion link (issue #1216) * No crop on thumbnail (#1217) * Use `get_thumbnail` to serve video thumbnail via caching system, to prevent video folder url to be publicly available (#1221) * Addition of a toolbar in the theme description editor of a chain. (issue #1185) * Correction enabling channels and themes to be assigned to a set of videos (issue #1106) * Manage restricted video access right in playlist ## Accessibility improvements: (#1219) * Modify some redundant title strings, adding the targeted object * Remove some redundant titles * Correct i18n strings * Correct duplicated id * Remove broken aria-label id * Add flatpage title in h1 for accessibility + remove H1 title in legal notice and accessibility statement pages ## Quality of Code: * Minor code formatting * Add missing DocStrings * Upgrade GitGuardian config version
Bonjour,
Certaines requêtes n'étaient pas adaptées pour mysql dans le script de migration pour BBB. J'ai donc ajouté une condition en fonction du type de base précisé dans la variable MOODLE_DB_TYPE
J'avais aussi l'erreur
TypeError: tuple indices must be integers, not strà cause de certaines chaines dans des tuples. J'ai mis le numéro d'index à la place.J'ai aussi ajouté une correction pour le module d'import de vidéo, en lien avec #1213, car cela empêchait d'importer les vidéos lors de l'utilisation de l'option --use-import-video avec un BBB < 2.3