Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
branch: master
Commits on Oct 14, 2014
  1. @maneframe

    bring in blackboard files without image extentions

    maneframe authored
    turns out some blackboard files don't actually
    have an extension, meaning we were ignoring them
    when they were actually images
    
    test plan:
    * import the package referenced by the ticket
    * should have images in assessment questions
    
    closes #PFS-23
    
    Change-Id: I06d2ef68bd10113e4841bc07a1641724de1265ef
    Reviewed-on: https://gerrit.instructure.com/42601
    Reviewed-by: Dave Jungst <dave@instructure.com>
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
    Product-Review: Jeremy Stanley <jeremy@instructure.com>
    QA-Review: Sam Derby <sderby@instructure.com>
Commits on Feb 28, 2014
  1. @jstanley0

    escape InstructureMetadata XML inserts

    jstanley0 authored
    fixes CNVS-11196
    
    test plan:
     - create a new course
     - create a question bank in this course that has
       an ampersand in its name, for example: "this & that"
     - create a question in this question bank
     - copy the course
     - in the copied course,
       - the question bank name should retain the ampersand
         ("this & that", not "this  that")
       - the question text should not gain spurious "div" and "p" text
         ("Test question", not "divpTest Question/p/div")
    
    Change-Id: I7934a036f9f96c83ae013c1881736a2f579c40a3
    Reviewed-on: https://gerrit.instructure.com/30267
    Tested-by: Jenkins <jenkins@instructure.com>
    QA-Review: Matt Fairbourn <mfairbourn@instructure.com>
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
    Product-Review: Bracken Mosbacker <bracken@instructure.com>
Commits on Jan 31, 2014
  1. @maneframe

    only send to saxify if element tree is parsed correctly

    maneframe authored
    Change-Id: I70fcd82d85f5b318f803b32a351c55f9fcec6ebb
    Reviewed-on: https://gerrit.instructure.com/29068
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
    Product-Review: Bracken Mosbacker <bracken@instructure.com>
    QA-Review: Matt Fairbourn <mfairbourn@instructure.com>
Commits on Jan 22, 2014
  1. @bracken @codekitchen

    Revert "lxml fix for webct and d2l_2p0 namespaces"

    bracken authored codekitchen committed
    This reverts commit 9de5189.
    
    Change-Id: I8c659983429dc8eb1773ef0199f003dc6a9d064f
    Reviewed-on: https://gerrit.instructure.com/29090
    Tested-by: Brian Palmer <brianp@instructure.com>
    Reviewed-by: Brian Palmer <brianp@instructure.com>
    QA-Review: Brian Palmer <brianp@instructure.com>
    Product-Review: Brian Palmer <brianp@instructure.com>
  2. @maneframe

    lxml fix for webct and d2l_2p0 namespaces

    maneframe authored
    Change-Id: I54e70edfa8bfd13d9ce9cfa221af1a4bc7d4c8c6
    Reviewed-on: https://gerrit.instructure.com/28842
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
    QA-Review: Matt Fairbourn <mfairbourn@instructure.com>
    Product-Review: James Williams  <jamesw@instructure.com>
Commits on Jan 18, 2014
  1. @bmad

    Revert "use lxml to recover from xml parsing errors"

    bmad authored
    This reverts commit 1ab0b24.
    
    Change-Id: Ib4221f3ea001326e6e8e9fe5cf547088d974a6d1
    Reviewed-on: https://gerrit.instructure.com/28851
    Tested-by: Bryan Madsen <bryan@instructure.com>
    Reviewed-by: Bryan Madsen <bryan@instructure.com>
    QA-Review: Bryan Madsen <bryan@instructure.com>
    Product-Review: Bryan Madsen <bryan@instructure.com>
Commits on Jan 17, 2014
  1. @maneframe @bracken

    use lxml to recover from xml parsing errors

    maneframe authored bracken committed
    requires the lxml python package
    
    installation instructions:
    (OSX) easy_install lxml
    (Ubuntu) apt-get install python-lxml
    
    fixes #CNVS-10411
    
    Change-Id: Ied85a00245623b843e020251fc78de0772cfcc35
    Reviewed-on: https://gerrit.instructure.com/28598
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
    QA-Review: Nathan Rogowski <nathan@instructure.com>
    Product-Review: Bracken Mosbacker <bracken@instructure.com>
  2. @maneframe

    allow ConditionVar elements to be directly under RespCondition

    maneframe authored
    fixes #CNVS-10487
    
    Change-Id: I3d50859c60e6f3ad0135370c8d646d40845e44cd
    Reviewed-on: https://gerrit.instructure.com/28607
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
    QA-Review: Matt Fairbourn <mfairbourn@instructure.com>
    Product-Review: James Williams  <jamesw@instructure.com>
Commits on Oct 4, 2013
  1. @smeredith0506

    global git repo arg handling

    smeredith0506 authored
    Change-Id: I04e020f850e3133b473dca74e5f93ce082766c61
    Reviewed-on: https://gerrit.instructure.com/23156
    Reviewed-by: Bryan Madsen <bryan@instructure.com>
    Product-Review: Bryan Madsen <bryan@instructure.com>
    QA-Review: Bryan Madsen <bryan@instructure.com>
    Tested-by: Jenkins <jenkins@instructure.com>
  2. @maneframe @bracken

    correctly process webct calculated question text

    maneframe authored bracken committed
    closes #CNVS-2168
    
    Change-Id: Iec527c12b9b9b0f23efc4cde09e6d0251dbec7cc
    Reviewed-on: https://gerrit.instructure.com/24660
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
    Product-Review: Bracken Mosbacker <bracken@instructure.com>
    QA-Review: Matt Fairbourn <mfairbourn@instructure.com>
    Tested-by: Jenkins <jenkins@instructure.com>
Commits on Jan 11, 2013
  1. @bmad

    fix hudson_setup.sh to work with new gerrit mirror url

    bmad authored
    Change-Id: Ib895694543619f2b8cf02594fb9d8aad51be5e2a
    Reviewed-on: https://gerrit.instructure.com/16728
    Reviewed-by: Bryan Madsen <bryan@instructure.com>
    QA-Review: Bryan Madsen <bryan@instructure.com>
    Tested-by: Bryan Madsen <bryan@instructure.com>
Commits on Dec 4, 2012
  1. @bracken

    pass label through to manifest node if available

    bracken authored
    some vendors use the label attribute instead of identifier
    as the unique id. This allows them to be processed correctly
    
    refs #CNVS-1346
    
    Change-Id: I007952a7c9a99b61056816cce1feb33ac1e72277
    Reviewed-on: https://gerrit.instructure.com/15818
    Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
    Tested-by: Jenkins <jenkins@instructure.com>
Commits on Aug 16, 2012
  1. @zachpendleton @bracken

    gracefully handle response_labels w/o ident attribute.

    zachpendleton authored bracken committed
    fixes #9876
    
    some malformed imports are missing the ident attribute
    on some response_labels. this commit returns early instead
    of throwing an exception.
    
    Change-Id: I793172a8d343ec11436aec0ff2249feba3f94dcb
    Reviewed-on: https://gerrit.instructure.com/12951
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
Commits on Jun 25, 2012
  1. @bracken

    make sure imsmanifest.xml is processed first in a directory

    bracken authored
    refs #8051
    
    Change-Id: I82c2a9827a10e352e57b3c0f7168987843786c32
    Reviewed-on: https://gerrit.instructure.com/11822
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Simon Williams <simon@instructure.com>
Commits on Jun 21, 2012
  1. @bracken

    pull item references from selection metadata

    bracken authored
    refs #8862
    
    Change-Id: I977694e2e0bfd702926a9247f505fc5bf56c02c1
    Reviewed-on: https://gerrit.instructure.com/11742
    Reviewed-by: Simon Williams <simon@instructure.com>
    Tested-by: Jenkins <jenkins@instructure.com>
Commits on Jun 1, 2012
  1. @bracken

    flag text question responses as text

    bracken authored
    Previous to this text responses would be converted in the
    same way as html responses, so a QTI importing tool wouldn't
    be able to tell the difference. This now flags them as 'text'
    
    refs #8771
    
    Change-Id: Id217e6f5da5c80728e187ad7fc5bcb3840bc2492
    Reviewed-on: https://gerrit.instructure.com/11202
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Simon Williams <simon@instructure.com>
Commits on May 30, 2012
  1. @bracken

    fix the outcome weight identifier referenced

    bracken authored
    refs #8143
    
    Change-Id: Iba59f05b7096e61ae3700bbbeb4c09fadd951700
    Reviewed-on: https://gerrit.instructure.com/11118
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Simon Williams <simon@instructure.com>
Commits on May 29, 2012
  1. @bracken

    copy files referenced in superfluous manifest files

    bracken authored
    some vendors create a manifest file for each question and
    related file, this makes it so that the files referenced
    in there are copied and become part of the single manifest
    when finished. It also grabs the expected output name from
    the webct meta data.
    
    refs #8051 #8671
    
    Change-Id: I0e3f3211123dd5301335538f5f332717cc457746
    Reviewed-on: https://gerrit.instructure.com/10936
    Tested-by: Jenkins <jenkins@instructure.com>
    Reviewed-by: Simon Williams <simon@instructure.com>
Commits on Apr 18, 2012
  1. @bracken

    pass selectionExtension properties through

    bracken authored
    refs #8159
    
    Change-Id: I52c211bc9d0e6e64aaa63866aed9c29cb355097c
    Reviewed-on: https://gerrit.instructure.com/10117
    Tested-by: Hudson <hudson@instructure.com>
    Reviewed-by: Simon Williams <simon@instructure.com>
Commits on Apr 2, 2012
  1. @bracken

    don't try to write a response condition if there isn't one

    bracken authored
    refs #6612
    
    Change-Id: I80bbd6a1e4936375d888b989a303c75d984c772d
    Reviewed-on: https://gerrit.instructure.com/9746
    Tested-by: Hudson <hudson@instructure.com>
    Reviewed-by: Cody Cutrer <cody@instructure.com>
Commits on Feb 2, 2012
  1. @bracken

    if an identifier is set on assessment use it

    bracken authored
    the filename was always being used as the identifier
    for an assessment. That should have been the fall-back
    identifier instead of the default.
    
    Test Plan:
     * run canvas-lms plugin specs
    
    refs #7028
    
    Change-Id: If928d8178ccba7a29980ba8019ba7b0cf5681bc2
    Reviewed-on: https://gerrit.instructure.com/8422
    Tested-by: Hudson <hudson@instructure.com>
    Reviewed-by: Cody Cutrer <cody@instructure.com>
Commits on Jan 6, 2012
  1. @bracken

    don't throw an exception for a metadata field out of place

    bracken authored
    Some vendors who generate QTI put all the same meta data values
    in all the metadata places. This isn't valid, but can just be
    ignored instead of killing the whole converstion process
    
    closes #6807
    
    Change-Id: I60e6d2eda99d463198e9c40d3636410f7eedb9a5
    Reviewed-on: https://gerrit.instructure.com/7918
    Tested-by: Hudson <hudson@instructure.com>
    Reviewed-by: Brian Palmer <brianp@instructure.com>
Commits on Nov 15, 2011
  1. @bmad

    correctly remove vendor/plugins/qti_migration_tool in hudson build

    bmad authored
    Change-Id: Idabe3c04c0e23645a691ac9fc7e69d5e34896b46
    Reviewed-on: https://gerrit.instructure.com/6990
    Tested-by: Hudson <hudson@instructure.com>
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
Commits on Nov 1, 2011
  1. @bracken

    copy any extra resources when converting

    bracken authored
    Files that were listed in the manifest weren't getting
    copied over to the qti 2.0 output. This copies all those
    files and puts them in a sinle resource object in the
    output manifest file
    
    Change-Id: Ie01055da7a42dd5783465ac215f11a751b0398ce
    Reviewed-on: https://gerrit.instructure.com/6101
    Reviewed-by: Brian Palmer <brianp@instructure.com>
    Tested-by: Bracken Mosbacker <bracken@instructure.com>
Commits on Oct 30, 2011
  1. @jtolds

    removing debian stuff from this branch

    jtolds authored
    Change-Id: Ib789a4caf9833e7008359e134bab915e2e20ea6d
    Reviewed-on: https://gerrit.instructure.com/6579
    Reviewed-by: JT Olds <jt@instructure.com>
    Tested-by: JT Olds <jt@instructure.com>
Commits on Oct 27, 2011
  1. @bmad

    Add hudson_setup.sh for a hudson build of this plugin

    bmad authored
    Change-Id: I79f662e5b192b625aeffdfc885f6492d31a9327a
    Reviewed-on: https://gerrit.instructure.com/6243
    Tested-by: Hudson <hudson@instructure.com>
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
Commits on Oct 26, 2011
  1. @bracken

    make sections within a bank their own banks

    bracken authored
    refs #5857
    
    Change-Id: Id0748810e95427c805a4f5c3b92623a7e1afbf62
    Reviewed-on: https://gerrit.instructure.com/6497
    Reviewed-by: Cody Cutrer <cody@instructure.com>
    Tested-by: Bracken Mosbacker <bracken@instructure.com>
Commits on Oct 15, 2011
  1. @jtolds

    new deb: canvas-lms-qti-migration-plugin_0.1-10 professional

    jtolds authored
    Change-Id: I72b43b209bed850a53505af1357fe922a2b066f5
Commits on Oct 6, 2011
  1. @bracken

    allow the generic and bb-specific questions types to be set

    bracken authored
    Change-Id: If659310c9f0398e135804cc06d5edf933e1daf37
    Reviewed-on: https://gerrit.instructure.com/6010
    Tested-by: Bracken Mosbacker <bracken@instructure.com>
    Reviewed-by: Brian Palmer <brianp@instructure.com>
  2. @bracken

    make referenced ids have correct identifier

    bracken authored
    Change-Id: I7c64de5d9665dddae84b29bf802b65f8cd98baef
    Reviewed-on: https://gerrit.instructure.com/6001
    Tested-by: Bracken Mosbacker <bracken@instructure.com>
    Reviewed-by: Brian Palmer <brianp@instructure.com>
Commits on Oct 1, 2011
  1. @jtolds

    new deb: canvas-lms-qti-migration-plugin_0.1-9 professional

    jtolds authored
    Change-Id: Idfbae8ca53c93f996c25953540d5cc0415de7f18
Commits on Sep 23, 2011
  1. @bracken

    properly set altered item identifier

    bracken authored
    The identifier for an item would get an "ID_" added if it
    didn't start with a valid character, but this new id wasn't
    getting set for all references to this item
    
    refs #4153
    
    Change-Id: Ib770a2777de8207b2b0899dfa9759d04f395b3a0
    Reviewed-on: https://gerrit.instructure.com/5769
    Tested-by: Bracken Mosbacker <bracken@instructure.com>
    Reviewed-by: Brian Palmer <brianp@instructure.com>
Commits on Sep 21, 2011
  1. @jtolds

    new deb: canvas-lms-qti-migration-plugin_0.1-8 professional

    jtolds authored
    Change-Id: I14226f638c0abac93d0abe7e7e056d8f7c2bb67f
Commits on Sep 20, 2011
  1. @bracken

    added some common cartridge meta data fields

    bracken authored
    refs #4153
    
    Change-Id: Id2750d489073a9a4494fcd593f09f777f3038d5c
    Reviewed-on: https://gerrit.instructure.com/5725
    Reviewed-by: Brian Palmer <brianp@instructure.com>
    Tested-by: Bracken Mosbacker <bracken@instructure.com>
Commits on Sep 8, 2011
  1. @codekitchen

    add support for translating respondus FIMB questions, refs #5386

    codekitchen authored
    Change-Id: I0a9f6e277810c65f41d70122dec254f6cc3366dc
    Reviewed-on: https://gerrit.instructure.com/5489
    Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
    Tested-by: Brian Palmer <brianp@instructure.com>
Something went wrong with that request. Please try again.