remove unneccessary collection data fix up

fixes CNVS-11620

test plan
 - run migrations
 - should not error

Change-Id: I63e137f3c3154d1c0b64d2bbfd110a26f748a1f2
Reviewed-on: https://gerrit.instructure.com/31304
Reviewed-by: Cody Cutrer <cody@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
Product-Review: Rob Orton <rob@instructure.com>
QA-Review: Rob Orton <rob@instructure.com>
don't add verifiers to locked file links

test plan:
 - have a course with a public syllabus
 - link to two files in the syllabus body:
   - a locked file
   - an unlocked file
 - access the course syllabus without logging in
   - the unlocked file link should work
   - the locked file link should not

fixes CNVS-11569

Change-Id: I75d25a46c7e4fac43c7f187a4aba166be85d1010
Reviewed-on: https://gerrit.instructure.com/31270
Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
Product-Review: Bracken Mosbacker <bracken@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
QA-Review: Nathan Rogowski <nathan@instructure.com>

Conflicts:
	spec/lib/user_content_spec.rb
fix bug where essay questions answers not saved

closes CNVS-11603

test plan:
  - as an instructor
    - create and publish a quiz that:
      - has an essay question
      - has a time limit
      - allows students to see their submitted answers
  - as a student
    - start quiz
    - enter something via the RCE for the essay question (dont submit)
    - allow time to expire on the quiz
    - click 'Ok, fine' in the 'Times up' modal dialog that pops up
  - your answer should be displayed in the following quiz summary page

Change-Id: I0f8bc7ff177f3cd012ec8311fad631ec2f519348
Reviewed-on: https://gerrit.instructure.com/31403
Reviewed-by: Derek DeVries <ddevries@instructure.com>
QA-Review: Myller de Araujo <myller@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
Product-Review: Derek DeVries <ddevries@instructure.com>
only allow deleting enrollments from course in path

fixes CNVS-11685

test plan
 - delete an enrollment using the api
 - should only allow deleting from the specified course

Change-Id: I5ae0754b0322a2f6fa0817ce15e41a885c88cf2d
Reviewed-on: https://gerrit.instructure.com/31468
Reviewed-by: Cody Cutrer <cody@instructure.com>
Tested-by: Jenkins <jenkins@instructure.com>
QA-Review: August Thornton <august@instructure.com>
Product-Review: Rob Orton <rob@instructure.com>
override Quiz#context_module_tags to find both content_types

test plan:
 - have a quiz that was added to a module prior to the
   Quizzes::Quiz refactor
   (or manually update the ContentTag's content_type to 'Quiz')
 - set up a completion requirement (must submit, or must score)
 - verify that a student can complete the requirement

fixes CNVS-11613

Change-Id: I2bb5271e6b263173d98c9df8913044640698e7fb
Reviewed-on: https://gerrit.instructure.com/31424
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Derek DeVries <ddevries@instructure.com>
QA-Review: Matt Fairbourn <mfairbourn@instructure.com>
Reviewed-by: Bracken Mosbacker <bracken@instructure.com>
Product-Review: Bracken Mosbacker <bracken@instructure.com>
treesame commit of origin/stable/2014-03-01

Treesame-Commit-Id: 5bcb64223abddca24819285de551e35350ab90b9