CVS log for loncom/homework/grades.pm

Up to [LON-CAPA] / loncom / homework

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

- Get tinyURL <=> symb mapping, from course's nohist_linkprot_passback.db,
  written when a particular deep-link is first followed, in preference to
  current record in tiny.db (which cloning can change).

- Use a unique time-based key when storing a passback item as a hash value in
  linkprot_passback_pending.db in a course's data directory for scores which
  were not successfully sent to launcher CMS.

- Bug 6907. "Content in a course can be set to be deep-link only".
  Pass scores back to launcher CMS when clicker scores are uploaded, if
  accessed via deep-link with LTI-mediated link protection (and passback
  expected).

- Bug 6907. "Content in a course can be set to be deep-link only".
  Pass scores back to launcher CMS for collaborators on essayresponse items,
  when course personnel changed scores. (Requires each collaborator to have
  accessed via deep-link with LTI-mediated link protection.

- Change in args for &get_lti_score() in lonhomework.pm.  Third arg is now
  the section of the student for whom the score is being retrieved.

- Package variable; $registered_cleanup
  - set to 0 when handler() is called
  - set to 1 if routine to be called in PerlCleanupHandler phase added.
  - @Apache::grades::ltipassback used to store calls added by  grades.pm
  - @Apache::grades::lonhomework used to store calls added by lonhomework.pm
  - Most code in &do_ltipassback() in lonhomework.pm moved to &run_passback()
    to facilitate reuse.

- Fix indents and add missing closing curly brace.

- Bug 6907. "Content in a course can be set to be deep-link only".
  Use correct value for linkprot in item hash.

- Bug 6907. "Content in a course can be set to be deep-link only".
  Pass scores back to launcher CMS for students who accessed via deep-link
  with LTI-mediated link protection, when course personnel changed scores
  via: (a) "Select individual students to grade", (b) "Grading table",
       (c) "Grade page/folder for one student", or (d) " Upload Scores".

- Typo.

- For 2.11
  Backport a change in 1.627 omitted from earlier backport (1.596.2.12.2.51)

- Bug 6907. "Content in a course can be set to be deep-link only".
  Course personnel with mgr priv can pass scores back to launcher CMS for
  students who accessed via deep-link with LTI-mediated link protection.

- For 2.11.4 (modified)
  Include changes in 1.794, 1.795

- For 2.11
  Backport 1.789, 1.790

- For 2.11
  Backport 1.791, 1.792, 1.794, 1.795

- Fewer calls to &Apache::lonlocal::locallocaltime().

- Remove trailing whitespace.

- For 2.11.4 (modified)
  Include changes in 1.793

- For 2.11.4 (modified)
  Include changes in 1.745, 1.746, 1.747, 1.749, 1.793 (part)

- Bug 6979
  - Shortcut to student's view of problem etc. from 'View As' item in Functions
  menu above problem.
  - Part of code in &submission() subroutine in grades.pm moved to separate
  subroutine -- &show_last_submission() which can be called by &view_as_user(),
  which in turn is called from lonhomework.pm

- For 2.11.4 (modified)
  Typo in rev. 1.596.2.12.2.60.2.2

- For 2.11.4 (modified)
  Include changes in 1.791, 1.792

- Bug 6966

- Only need determine timestamp once per version.

- For 2.11.4 (modified)
  Include changes in 1.789, 1.790

- Ignore empty and commented out rows in bubblesheet format file.

- Always remove line feeds from lines returned by get_scantronformat_file()
  so chomp() no longer needs to be included when using the output.

- For 2.11
  Backport 1.788

- Bug 6968
  - Support bubblesheet grading of named or CODEd exams for exam folder
  containing sub-folders for which randompick and/or randomorder in use.

- For 2.11
  Backport 1.787

- In &updateGradeByPage() routine, need to declare %queueable and replace
  scalar used for partID. (fix changes in rev. 1.786).

- For 2.11
  Backport 1.785, 1.786

- Bug 6965. Removal of a problem from grading queue ignore awarded status
  of parts with non-handgradeable responses.

- No "Grading" link in breadcrumbs when not in course context.

- For 2.11
  Backport 1.782, 1.783, 1.784

- Accommodate access via "Grade ungraded submissions" link which doesn't
  prompt instructor to select Access Status.

- Accommodate access via "Grade ungraded submissions" link which doesn't
  prompt instructor to select Sections.

- Be wary of uninitialized package variables in perl modules under mod_perl.

- For 2.11
  Backport 1.780, 1.781

- Missing commas in 1.780

- "Send Messages" and "Grading Increments" rows only shown in display options
  table, if user has mgr priv.
- Save button on "Grading table" and "Grade page/folder for one student" screens
  grayed out unless user has mgr priv.
- "Upload Scores", "Process Clicker", and "Grade/Manage/Review Bubblesheets"
  items only included in "Automated Grading" panel if user has mgr priv.

- For 2.11
  Backport 1.777, 1.778, 1.779

- Move &css_links() from grades.pm to loncommon.pm to facilitate reuse.

- Bug 6943 <link> tag for css not needed if View Problem Text set to "no".

- Bug 6943
  Load CSS files for resources being displayed in Grading interface, i.e.,
  when "View Problem Text" is checked for "one student" or "all students".

- For 2.11
  Backport 1.776

- Fewer localized phrases needed.

- For 2.11
  - Wording changes in recognition that 'Download submissions' currently only
    applies to download of files submitted to an assignment.

- For 2.11
  Backport 1.775

- In "Download submissions" context, table listing parts/responsetypes
  not shown if none set up as dropboxes.

- For 2.11
  Backport 1.598, 1.599, 1.600, 1.601, 1.602, 1.603, 1.605, 1.607, 1.608,
           1.609, 1.610, 1.611, 1.612, 1.613, 1.614, 1.615, 1.616, 1.617,
           1.618, 1.619, 1.620, 1.621, 1.622, 1.623, 1.624, 1.625, 1.627,
           1.628, 1.629, 1.630, 1.632, 1.633, 1.634, 1.636, 1.637, 1.638,
           1.763, 1.764, 1.766, 1.767, 1.768 (part), 1.769, 1.771, 1.772,
           1.773, 1.774

- White space changes only.

- Grading interface changes March-April 2010 (rev. 1.598-1.633) reviewed.
  - Complete the elimination of form.handgrade
  - Re-instate use of &showResourceInfo()
    - If problem has multiple parts, can select which to grade
    - If problem has file upload items (in essayresponse) in multiple parts
      can select which parts to include when downloading submitted files
  - Keywords list menu only shown if selected problem contains one or more
    essayresponse items.
  - Table of options for grading individual students:
    (a) Send Messages - include ability to send a message (default is "Yes"
        if selected problem contains an essayresponse item, "No" otherwise.
    (b) Check For Plagiarism - only shown if selected problem contains
        an essayresponse item

- Keyword highlighting.
  Convert object to string to permit use of replace().

- Problem selector for "Content Grading" screen only needs a radiobutton
  for each resource.
- Order of args passed to lonstathelper::problem_selector() changed.
  smallbox, onclick args not passed by any callers of the routine --
  added for lonquickgrades.pm rev. 1.75, but use discontinued in
  rev. 1.84).

- Bug 6215
  Bubblesheet grading by a role (instructor or custom) with specific section

- coding style -- grep().

- Remove HTML markup from phrases for localization.
- White space changes.

- White space changes to improve readability
- Remove duplicate parentheses and surplus single quote.

- White space changes only (for readability).

- "Grading tabs" only in course context.

- Consistent punctuation in grading links

- Update comments.

- For 2.11
  Backport 1.762

- Parameter setting via table mode for users with both student and non-student
  roles.
- Pop-up launched via "Select User" link shows appropriate table(s) of users.

- For 2.11
  Backport 1.754, 1.755, 1.756, 1.757

- For 2.11
  - Backport 1.761

- Bug 5895

- For 2.11.2 (modified)
  - Include changes in 1.596.2.12.2.47

- For 2.11
  Backport 1.760

- Support upload of session data for iClicker 7 (also named iClicker Classic).

- For 2.11
  Backport 1.753 (part). Missed in earlier rev. 1.596.2.12.2.45.

- For 2.11
  - Backport 1.753

- Fix typo
- Standard indentation is four spaces.

- For 2.11.2 (modified)
  Include changes in 1.758

- Remove redundant line
- Update documentation

- For 2.11.2 (modified)
  Add closing brace and change to location of get_scantronformat_file()
  missing from 1.754 changes included in 1.596.2.12.2.41.2.3

- For 2.11.2 (modified)
  Include changes in 1.754, 1.755, 1.756, 1.757

- Uploaded bubblesheet data can consist of comma separated values.
  Wording changes and display style for bubbletype div ste to "block".

- Uploaded bubblesheet data can consist of comma separated values.
  - Domain configuration for conversion of csv file to txt file:
    (a) Delete first line (column titles) Y/N
    (b) Remove leading white space except question responses Y/N
    (c) Pad PaperID with leading zeroes, up to PaperIDlength Y/N

- Uploaded bubblesheet data can consist of comma separated values.
  - Set each value in each question field to correct length, and appropriate
    value, either letter A-J, or number 1-9 then 0, or '    1      ', as
    determined by selected data format.

- Uploaded bubblesheet data can consist of comma separated values.
  - Data will be converted to one of formats specified in bubblesheet
    format file, prior to saving in scantron_orig_* file.
  - Domain configuration for bubblesheet data format allows order of
     columns in csv data to be specified.
  - &get_scantronformat_file and &get_scantron_config moved from grades.pm
    to lonnet.pm to facilitate re-use.

- "Check For Plagiarism" now supported for submissions to Simple Problem
(Essay-open ended). Similarity checking only for submissions in same course

- For 2.11.2 (modified)
  Include changes in 1.751 and 1.752

- For 2.11
  - Backport 1.751, 1.752.

- Explicitly declare use of Apache::lontexconvert().

- In grading interface render tex within <m> </m> tags when displaying a
  student's submission to an essayresponse item.

- For 2.11
  Backport 1.742

- Three new args added to &getclasslist() to support filtering by submission
  status.
- Download Submissions selection for Sections, Groups, Access Status, and
  Submission Status now used to populate list of students for use by
  multidownload.pl

- For 2.11.2 (modified)
  Include changes in 1.748

- For 2.11
  Backport 1.748

- Bug 6754 LON-CAPA as LTI Consumer
  - For external tools set to be gradable, grading table from
    &displaySubByDates() includes values awarded instead of submissions.

- Set values for colspans in grading table which will support the number
  of stores_ items for a part (excluding awarded and solved).

- If &get_last_submission() or displaySubByDates() finds nothing recorded,
  for external tool report 'no grade passed back', instead of 'nothing
  submitted'.

- Call regex in scalar context when setting value for $is_tool.

- Bug 6754 LON-CAPA as LTI Consumer
  - Grading screens can be used to display passback transactions, and
    for override of scores etc., for external tools set to be "gradable".

- Unused scalar.

- Bug 5977.
  - Score upload form support for identification of a user based on clicker ID,
    for Course Coordinators who prefer not to use LON-CAPA's in-built
    "Process Clicker" utility.

- For 2.11.2 (modified).
  - Include changes in rev. 1.742.

- For 2.11
  - Backport 1.741.

- Sanity checking

- Support <randomlist> around parts in bubblesheet grading with CODEs.

- For 2.11
  - Backport 1.740

- Facilitate work of localization phrasechecker script.

- For 2.11
  - Backport 1.739

- Fix typo.

- Fix typo in 1.596.2.12.2.37.

- For 2.11
  - Backport 1.738

- Bug 6440.
  - Submission status can be used as filter for grading table.
  - Group(s) can be used as filter for grading table.

- For 2.11
  - Backport 1.736

- Score upload form supports identification of a user based on clicker ID,
  for Course Coordinators who prefer not to use LON-CAPA's in-built
  "Process Clicker" utility.
- clickers.db file on a library server contains key = value pairs, where key
  is (escaped) clicker ID, and value is (escaped) comma-separated list of
  usernames who registered that particular clicker ID.
- bi-nightly run of searchcat.pl will update clickers.db file.

fixed bug 6782, and escaped most localized messages used in Javascript blocks to make sure bugs like that do not happen again

- For 2.11
  Backport 1.735.

- Bug 6476
  Whether rndseed in use is stored in submission record depends on whether
  LON-CAPA 2.10.X or 2.11.X was hosting user's session. Check for question type
  when deciding if 'New variation this try' should be show for a particular
  transaction,  for both user's Previous Tries pop-up and for CC's grades screen
  (dates and submissions option).

- For 2.11
  Backport changes in 1.734, and one line from changes in 1.640 missed in
  earlier 1.596.2.2 backport.

- Bug 6476
  rndseed in use now always stored as part of submission record.

- For 2.11
  Backport 1.731, 1.732, 1.733

Bug 6476
  - Revert part of changes in grades.pm rev. 1.732
  - When the random seed is stored, store rawrndseed (the pre-processed random
    seed) in resource.$part.rndseed, since the pre-processed seed is the one
    needed to reconstruct the problem in Authoring Space.
  - Retains backwards-compatibility with data stored in 2.10 and 2.11, because
    the previous tries display, and submission history in the grading screens
    for questiontype of randmizetry both expect resource.$part.rndseed in the
    submission history (and for randomizetry the rawrndseed and rndseed are
    the same).

- Bug 6476
  - Store rawrndseed for parts with all responsetypes.
  - Eliminate storage of rndseed, since rawrndseed is the item needed to
    reconstruct the randomization for a problem in Authoring Space.
  - If questiontype is randomizetry rawrndseed and rndseed are the same,
    so use rawrndseed from the submission history, where it exists.
  - Support use of randomizetry for script blocks outside a part block in
    problems with a single <part></part> tag.
  - Support use of randomizetry for script blocks inside part blocks.
Note: randomizetry does not apply to randomization in script blocks
outside part blocks for problems with multiple parts, because the
number of tries is not uniquely defined.

- If the same student ID is used more than one student in the classlist,
  the username:domain belonging to the student with Active status is used.

- For 2.11
  - Backport 1.730

- For 2.11
  - Backport 1.726, 1.727, 1.728

Reverting changes concerning essay response in order to
improve display in hand-grading screen.

- Support for submission of files to externalresponse.
  - New routine: &portfiles_versioning() in lonnet.pm called after grading
    externalresponse to handle versioning of submitted files in portfolio.
  - &get_next_version(), &version_selected_portfile(), &file_name_version_ext()
    moved from grades.pm to lonnet.pm to facilitate re-use.

- Bug 6740 Out-of-order recording of submissions (by time).
  - Check for new transactions made immediately before call to lonnet::cstore()
    &Apache::inputtags::hidealldata() called if correct then incorrect,
    where awarded >= 1 when correct (feedback on correctness enabled).
  - Check for transactions made immediately after call to lonnet::cstore()
    if reply from lond::store_handler() is delay:N (where N s the number of
    transactions between the last retrieved in &initialize_storage() and the
    last stored immediately before permanent storage of the current transaction.
    &Apache::grades::makehidden() called if correct then incorrect,
    where awarded >= 1 when correct (feedback on correctness enabled).

- Bug 6740 Out-of-order recording of submissions (by time).
 - "reset_status" event for a problem part subsequent to out-of-order
   ("incorrect_attempted" post-"correct_by_student") eliminates
   need to provide "Hide" checkbox in grading menu for that particular
   transaction.

- Bug 6740.

- For 2.11
  - Backport 1.724, 1.725

- Remove line commented out in rev. 1.724.

- Bug 6726.
  - Delay adding "Draft Copy" text, if draft, and conversion of line feeds
    to <br /> when showing Submitted Answer in grading interface, for
    essayresponsa,e until after entity encoding (grades.pm rev. 1.720 change).

- For 2.11
  - Fix typo in rev. 1.596.2.12.2.28.

- For 2.11
  - Backport 1.711 (part), 1.712, 1.713, 1.715, 1.716, 1.717, 1.718, 1.719,
    1.720, 1.721, 1.722.

- Wording change.

- Space before self-closing tag.

Internationalization: Added missing &mt() calls

Changes regarding the display of students' submissions in handgrading-screen (Bug 6694)

Actually apply desired wording improvement from grades.pm 1.718
("- Inform about scope: essay only")

Corrections and improvements to Keyword Highlighting:
- Corrected unbalanced tags
- XHTML
- (simliar to) standard layout
- Separate fieldset for functions
- Inform about scope: essay only
- Internationalization

- Internationalization - added missing &mt() calls
- Error style for error message

Bug #5409:
- Also respect customresponse (multiple input fields possible, too)
- Display of answer after "Submitted Answer:" on grades screen

(Co-op with Stefan Droeschler and Marcel Kruse)

Fix documentation typos

- For 2.11.
  - Backport 1.714.

- Correction to change in rev. 1.691.
  - Bubble line number in corrections interface was being incorrectly
    incremented by 1 for problems with multiple responseitems in cases
    where neither randompick nor randomorder were in use.

Fix typos in rev. 1.712

Improve internationalization:
- Proper compilation of output string, then pass to translation
- Re-use existing phrases

Avoid "scantron" (-> "bubblesheet")

- For 2.11
  - Backport 1.705, 1.706

- For 2.11
  - Backport 1.710

Improve bubblesheet grading upload process - Better overview, consistent layout:
- Standard design using confirm_success
- Additionally show absolute matching results
- Skip warning and additional info about low percentage of matches in case of 100% matching.
- Consistent punctuation, consistent wording

- Updated related translation file entries
- de.pm: translation corrections and improvements

- For 2.11
  - Backport 1.707, 1.708, 1.709

Improved wording using quant
Updated entries and added missing entries to German translation file

Do not translate twice

Bubblesheet grading - mismatch tables:
Improve display improvement in rev. 1.658: also correctly display _multiple_ blanks to allow proper comparison of mismatching grading data

- For 2.11
  - Eliminate duplicate declaration (caused by changes in rev. 1.596.2.12.2.16).
  - Eliminate unused hash.

- Remove unused hash.

- For 2.11
  - Exclude HTML tags from phrase to translate.

- Quoting change to &mt() call.

- For 2.11
  - Backport 1.702 (part -- excluding white space changes).
  - Fix typo -- missing closing paren in 1.596.2.12.2.20.

- For 2.11
  - Backport 1.703, 1.704.

- Fix typo
- Improve comment wording

- Internationalization: Added missing &mt() calls
- Fixed some minor documentation typos

- For 2.11
  - Backport 1.701
  - Exclude HTML tags from phrases to be translated (commensurate with changes
    in localize/localize/de.pm rev. 1.463.2.12).

Added box with links to by students submitted files for all four options in  Hand-Grading-Screen.
Direct file access used to be possible under the option "Last Submission" only.

Corrected typo in rev. 1.42:
Added missing blank to properly include onchange attribute for javascript

- For 2.11
  - Backport 1.694, 1.695, 1.696, 1.697, 1.698.

"Manual Grading by Page or Sequence":
- Improved and consistent page layout - Use pick_box for selection of grading options. Moved hidden fields outside pick_box to comply XHTML conformity.
- Internationalization: grading table header

Enhancements according to Bug #6656, regarding wording in grading screens.

Improved wording, punctuation and line breaks on "hand grading"-screen for uploaded submissions
and updated German translations.

Exclude HTML tags from translation phrases.
Update corresponding German translation file phrases and translations.

Exclude HTML tags from translation phrases.
Update corresponding German translation file phrases and translations.

Manual Grading

- Internationalization:
    - Added missing &mt() calls
    - Corrected plural switch
- XHTML:
    - Proper inclusion by corrected XHTML structure (handback file row and WGT input field)
    - Message icon and file type icon: alt attribute
    - lower case class attribute

XHTML: action attribute in <form> tag required (dummy attribute added)

- For 2.11.
  - Backport some additional changes in rev. 1.691 not included in original
    backport (1.596.2.12.2.16).

- For 2.11
  - Backport 1.689, 1.690, 1.691, 1.692, 1.693.

- Eliminate duplicate definition.
- Empty line before =cut in POD.
- scantron_find_student() subroutine is within this package.

- Bubblesheet grading of CODEd exams in which exam folder has either randomorder
  or randompick set causes new parameter (examcode) to be set at map level for
  a sepecific user each graded student.
- Parameter value is set to CODE on the student's bubblsheet exam.
- If used, requires course roles to be selected in a user session on a 2.11 server.
- Ensures resources are displayed in same order to student, and more importantly,
  ensures points totals will be correct in student's quickgrades view, if randompick
  was used.

- Bubblesheet grading  -- randompick and randomorder for folder being
  graded now supported in cases where questions in folder require
  different numbers of bubble rows.

- Bubblesheet grading
  - &get_response_bubbles() routine added in rev 1.447 ceased to be used
    from rev. 1.503 onwards following addition of &prompt_for_corrections().

- Printing Problems from folder for CODEd assignments:
  - If randompick parameter set for folder, seed used for selection of
    resources is set in lonnet::rndseed() based on CODE.
- Bubblesheet grading CODEd exams -- randompick for folder being graded
    now supported.
  - If randompick parameter set, seed used for selection of resources is
    set in lonnet::rndseed() based on CODE.

- Work in progress.

- For 2.11
  - Backport 1.688.

- When users with pav and vgr priv (e.g., Course Coordinators)  print a
  folder of resources for anonymous CODE(s) or for named student(s), e.g.,
  for a bubblesheet exam, parameter in map for hiddenresource will be
  ignored when retrieving resources for folder in random order.
- Supports use case where CC sets hidden for exam folder while printing
  bubblesheet exam, and also while grading exam.

- For 2.11
   - Backport 1.684 (part).

- For 2.11
  - Backport 1.685, 1.686, 1.687.

- Suppress "Apache2::RequestIO::print: (103) Software caused connection abort"
  messages in Apache error log by returning OK when internal redirect is used.

Consistent wording: "file name" vs. "filename"
Use "filename" since this is more commonly used.

Consistent wording: username:domain

"Exam" in general or LON-CAPA's "Bubblesheet Exam"? Improved wording to clarify and less confusion.

- For 2.11
  - Backport 1.679, 1.680, 1.681, 1.682, 1.683.

- Typo in 1.682.

- Localization: move HTML mark-up out of &mt().

- Localization -- Consistency between &mt() and corresponding key in de.pm.

- Fix quotes in rev 1.679.

- Localization -- remove escapes from phrases to translate.

- For 2.11
  - Backport 1.677, 1.678.

- For 2.11
  - Backport 1.675.

- Bubblesheet grading and verification of "CODEd" and "NAMEd" exams
  available for exams where randomorder parameter set for exam folder.
- Routine used to create array of resources in the order used in the exam
  -- either CODE-based, or NAME-based -- moved to routine: &users_order()
  to facilitate reuse.

- Bubblesheet grading CODEd exams -- randomorder for folder being graded
  now supported.
  - If randomorder parameter set, seed used for ordering of resources is
    set in lonnet::rndseed() based on CODE.

in Grading > Select individual students to grade > Compose message to student
improved the interface for composing a message:
- the table is now more compact
- removed deprecated table colors
- moved the include checkbox to the left to be more clear
- shortened the textfield 'subject' to fit in the popup

Use standard styles (info/warning/error):

- For 2.10.X branch
  - backport 1.674.

- For 2.11
  - Backport 1.674.

- Bug 5712
  - Reverse change in rev 1.426 which converted %old_essays from global to
    lexically local scope within &submission().

- For 2.11.
  - Backport part of changes in 1.649, ommitted in 1.596.2.12.2.1.
  - Bacport 1.673.

- Include $bubbles_per_row as additional arg in call to &graders_resources_pass()
  within &checkscantron_results().

- Backport 1.667.

- Backport 1.671, 1.607 (part), 1.609 (part).

- Backport 1.668, 1.670

- Backport 1.666.

- Backport 1.663

- Backport 1.662.

- Backport 1.649, 1.672.

- coding style -- less wrapping of lines in editor.

- Bug 6450. Course personnel with vgr priv can view student's prior
  versions of a Bridge Task (BT).
  - bridgetask.pm
    - generation of Show Slot list and Show Grading Status buttons moved
      to new routine: &add_slotlist_button().
    - "Show Slot list", "Show Grading Status", "Show a previously done version"
      "Get a submission to grade", "Change reservation" buttons not shown
      when course personnel display student's version of a BT.
    - ensure tags have unique ids (replace duplicate LC_task_feedback with
      LC_task_answer.

  - grades.pm
    - Header: "View of the problem" replaced with "View of the problem for
              <student name>
    - Header: "Correct answer" replaced with "Correct answer for "
              <student name>
    - New routine: &show_previous_task_version() to show student's previous
      version of a BT.
    - New routine: &choose_task_version_form() provides a web form to select
      display of a student's version of a BT (in same window or in pop-up).
    - New routine: &previous_display_javascript() javascript to launch pop-up
      etc. for other version.
    - &startpage() has four additional args: $stuvcurrent,$stuvdisp,$nomenu,$js
      student's current BT version, version being displayed, no page menu (for
      pop-up), and javascript to launch previous version pop-up.a

  - lonhtmlcommon.pm
    - &resource_info_box() has two additional args: $stuvcurrent,$stuvdisp
      student's current BT version, and version being displayed.

- Fix typo in rev. 1.577.

- Needed to satisfy W3C validation.
- lonquickgrades::endGradeScreen() already calls loncommon::end_page().

Bug #6165: leave it to the browser to decide whether or not to have scrollbars
when composing messages to students

Bug 6455 - Progress Indicator now jQuery. Still need to test across browsers.

Turning Technologies clicker support

- Backport 1.665.

- Fix to change in rev. 1.658.
  Interpolation indices in &mt() call needed to be changed following replacement
  of a single &mt() call for multi-phrase text with two &mt() calls
  (one for each phrase).

 Backport 1.664.

- Bubblesheet operator role can be selected again.

- Modify change in rev. 1.596.2.4 which was preventing display of resource
  info when showing result of grade assignments based on clicker file data.

Using bubblesheets to record grades for handgraded exams
  -- choose what filled last bubble represents (either a zero or max,
     i.e., 5 for 5 bubbles per line, 10 for 10 bubbles per line etc.)
grades.pm
  - New routine: &hand_bubble_option() determines if sequence to be graded
    contains response types which are handgraded, and offers radio buttons to
    choose score for last bubble.
response.pm
  - &repetition() routine can be called in scalar context of in array context.
    scalar: returns number of lines to encode weight
    array: returns (number of lines to encode weight, bubbles/line).
chemresponse.pm imageresponse.pm caparesponse/caparesponse.pm
  - &response::repetition() now explicitly called in scalar context.
Work in progress.
  (Corresponding option needed in lonprintout.pm to set
   correct number of bubble rows for handgraded questions with
   weight >= bubbles per row).

- Backport 1.661.
- Fix typo in backport in 1.596.2.6.

- Change to way lonnet::dirlist() reports an error condition (i.e., no_host,
  con_lost, no_such_dir, empty etc.) when retrieving directory list information
  with ls3, ls2 or ls.
  - error is passed as second item in a pair of return values.
  - first item in return values is a reference to an array of actual files.

Typo

- Backport 1.660.
- Backport parts of 1.630, 1.631 missed in earlier backport (in 1.596.2.4).

- Unbalanced <span> tag from surplus $grayFont.
- Eliminate unnecessary concatenation.

- Typo in backport of 1.648 (in 1.596.2.4).

- Backport 1.658, 1.659.

- Typo in backport of 1.642 (1.596.2.4).

- Remove extra word.
- Consistent wording: "bubble sheet" -> "bubblesheet"

Improvements to bubblesheet grading and related localization
- Warning style for warning messages
- Use monospace font for mismatch table to allow proper comparison of mismatching grading data
- Move html tags outside phrase to be translated
- Move dynamic data outside phrase to be translated to actually allow translation
- Correct unbalanced tags (<p>)
- Consistent wording "Bubblesheet"

- Backport 1.597 (part), 1.598 (part), 1.601 (part), 1.606 (part), 1.623 (part),
           1.627 (part), 1.630, 1.631, 1.632 (part), 1.638 (part), 1.642, 1.643,
           1.644, 1.646, 1.647, 1.648, 1.651, 1.652, 1.653, 1.654, 1.655, 1.656,
           1.657.

- Warnings for "point value larger than weight" accumulated in @warnings instead
  of @skipped, since these scores are saved and not skipped.
- Explicitly include "use" for Apache::bridgetask.

- Missing localization.

- Missing localization.

- Handback of multiple files to one student.
  - Streamline process (one message for all returned files for a problem).
  - Localize message text according to recipient's language.
  - More robust if multiple file uploads by student for a single response:
    - Handback files will be uploaded and message sent for each and all
      files returned by the instructor.

- Move html tags outside phrase to be translated.

- Add some missing localization.
- CSS styles for paths to handback files
- Some <br /> tags to improve readability

- Bug 5629, 6456. Hand-back of file(s) to a student.
- prevent duplicated uploads in multi-part problems with non-numeric part ids.

- 'Grading Menu' button was eliminated in rev 1.614.

- Additional (eighteenth field) BubblesPerRow in Bubblesheet Format file.
  - specify bubbles available for each question row (default is 10).
- Support case where question (or subquestion for optionresponse,
    matchresponse, or rankresponse) requires more bubbles than in one line.
  - Students prompted to bubble once in multiple lines.
  - Numbering recognizes multiple lines per question.
  - Also supports: imageresponse, stringresponse, formularesponse where
    students leave link(s) blank if weight assigned is greater than
    bubbles per line -- multiple lines also supported in this case.

For example:
- 15-bubble MSU Art History exams now supported on standard 10-bubble
    bubblesheets used at MSU.

Bug 5853: Don't use tradmarked brandname "scantron" in interface anymore.
Some rests have been found and changed to "bubblesheet":
- Updated related scripts
- Consistent wording
- Tidy up translation files: Synchronized existing entries with used phrases in the current scripts.
  If the translation of an updated phrase is unclear, the old translation is kept as a comment and the original English phrase is used instead.

Increase the width of the popup window in manual grading (message to student) to avoid hiding the right border of the textarea. Window found to be to small in Firefox 3.6 and 4.0

- handler should return OK or HTTP_NOT_ACCEPTABLE;
- check if user is in course context and/or has priviliges to view page.
  - internal redirect to /adm/quickgrades if lacking privileges.

Easier way to get back to the problem grading menu

Adapt to also work with new i>clicker2 (numerical and alphanumerical)

When picking a student who might correspond to an unregistered clicker,
show the clickers that the students registered including similarities.

- Backport 1.641

- Fix typos.
- Eliminate escaped quotes.
  - No localization phrases to update as keys are not currently in localize/*.pm files.

- Eliminate duplicate code.
- Consistent separator when rejoining @parts into string for key in version hash.

- Backport 1.640.

- New Question Type - randomizetry
- New Variation displayed after N tries (default N=1).
   - Additional args (type,trial,rndseed) for the following:
     &get_analyze(), &get_order(), &get_radiobutton_correct_foil(), &cleanRecord()
   - 'New variation this try' displayed in grading screen submissions view
     if try used different rndseed.
   - rndseed used shown in submission record.

Grading tab

* Undoing 1.634
- was causing JavaScript error
- formulation different from the other alerts (need consistent message)
- what the button actually says depends on the browser
* Warning message if points>weight (caused a problem this week for an instructor)

Bubblesheet operator upload works again

Icons for grading-interface.

- Backport 1.597 (part), 1.635 (part).

- Bug 6251.
  - Third arg is now a ref to a hash, instead of a hash.
  - New (optional) fourth arg, is an additional attribute for the
    select tag, e.g., an onchange action.
- Update calls to loncommon::select_form() accordingly.

- Eliminate surplus line following changes in 1.574.2.7.

Correct pop up error message.. no longer Browse button but Choose File button.

The logic of this completely escaped me.

Standard tables
Standardize page headers
Internationalization
Detect double data on clickers (after assignment of unknown clickers)

Example for the use of custom hints.

Once you had entered anything in the collaborator field, you could never get rid of it.

Need to do this the pedestrian way. The whole script is condensed into one
line, and the \\ <![CDATA otherwise comments out the complete script.

A good chunk of Bug #6250

Bugs #6258: take out of grading queue when uploading scores

Bug #4354: don't need this feature anyway.

Part ID is actually Response ID
Don't lose the symb!

Handgrading needs to be decided on response-level. Not on part-level, and
not with a problem-wide $env{'form.handgrade'}

$res_error needs to be defined
Response ID, not Part ID

Allow user to select probem when called context-free

Move submissions downloading one level up.

Forgotten breadcrumbs

- scantron -> bubblesheet in text displayed to users.

Continuing fixing up $symb-transfer

Direct jump from What's New into grading
Trying to reduce the historically grown number of states that the
grading interface can be in. This should be determined by privileges.

More breadcrumbs and "ungraded" mode

More breadcrumbs

More breadcrumbs

More breadcrumbs, remove unneeded code.

Eliminate unused "form.saveStatus"
Breadcrumbs

Fix call to scantron_selectphase

Generate headers again. Breadcrumbs still missing.

Undoing change to breadcrumbs

Saving my work ...

Unifying the way symbs are passed around

Currently broken. Just saving work for Gerd.

Changed the order of "Specify file and which Folder/Sequence to grade" and "Specify a bubblesheet data file to upload.". The normal workflow would be: first upload  the data an then use this uploaded data for grading. If you already uploaded a bubblesheet data, just skip the first box where to upload the data.

Cleaning up form.probTitle that was (or failed to be) moved all over the place

- Backport wording changes in rev. 1.584.
- Consistent wording.

- Backport 1.604.

- Remove line replaced (and commented out) in rev. 1.540.

Moving "verify receipt" one level deeper

Work in progress: cleaning up grades.pm

Clean out CBI "Token" mechanism (never used)
Use common routines for some grading menus
... work in progress, currently partly defunct

Just saving my work

Saving my pre-cleanup work

Saving my work: cleaning up grading interface. Currently defunct.

- &scripttag()-call for XHTML-conformity.
- Deleted non-opend but closed <b>Tags.

- Bug 6119 Anonymous Survey.
  - course personnel can not view both identity of submitter and submission details.

- Wording change.
  - remove extra .

Removed double spacing from translation phrases
 -> HTML/Browsers ignore them anyway
 -> less error prone for translators
Adjusted language files accordingly (de, es, zh)

- Prevent ISE.
 Can be triggered if a user has multiple tabs open switched course,
 then tries to view grading screen in originally selected course.

Optimized manual grading screen layout:
- Optimized floating of items in submission details option list
- Moved additonal functions (Activity, Change Parameters) directly below corresponding student's name; optimized spacing
- Each student has HTML headline now
- Moved alternating colors from student's name background to left side border
- No additional border needed for each student; adjusted margins and padding
- Better separate footer from rest
- Adding warning style to "...do not save the scores" message
- Added missing &mt() call
- Removed now unused styles (including changes in grades.pm 1.588 and grades.pm 1.590)

- Backport 1.586, 1.591.

- Early out from &scantron_get_maxbubble() to avoid ISE if no navmap object.

- Backport 1.582, 1.583.

Corrected unbalanced tags from rev. 1.588: Removed outdated starting div

XHTML: Lower case attributes (onClick, onChange, onMouseDown)

Screen layout consistency and floating enhancements:
- Replaced some special styles by standard boxes and headlines
(work in progress)

- Customization for GCI_3.
  - Block CC access to individual student submission data via
    grading screens in gcitest courses.

- Backport 1.576.

- Backport 1.575, 1.581, 1.587.

- Wording change.

- Sanity checking to prevent ISE.
  - Confirm $resource is an object before calling symb() method.

- Backport 1.580.

Manual Grading:
Use standard data_table for grading table

- Problem Part table:
    - Standard data_table
    - Added headlines
    - Translate problem type
    - font-weight normal
    - Removed unused variable $col
- "Receipt Number":
    - Consistent Wording
    - Updated &mt() calls and kept translation file entries (already up-to-date)
    - Added warning style to "no match" message

Amend 1.582 changes.
- Remove surplus &scantron_get_maxbubble() call
- Adding navmap object check for one more call to &scantron_get_maxbubble().

- Eliminate some ISE occurrences.
- Check attempt to create a navmap object resulted in an object.
  - In case of failure display an error message which includes a link to re-initialize the course.

Scantron is a Trademark

- Bug 6043.
- Fix typo from localization changes in 1.540.

- Backport 1.579.

- Remove escaped quotes from phrase to localize.

- Bug 5965. Fix typo introduced in backport in 1.528.2.11

- Reverse one change in backports included in 1.528.2.11.
  - Change to pass @resources by reference (in 1.554) to &grade_student_bubbles() should not be part of 2.8.X.

- Backport 1.578

- Backport 1.578

- Bug 5960
- Reversing part of rev 1.549. Use of [quant,_1,student] in the &mt() call is inappropriate, as the arg is a select box form element not a simple number.

Localization:
- Resolved translation issues with ID: Now use "Part ID" instead of just "ID"
- Optimized &mt usage for "Part", "Part ID" and other
  (Note: Values are also displayed in bold now. Redesign expected anyway.)
- Added missing &mt calls
- Added missing translations (more to do)

- Optimized nowrap
- de.pm: some other little updates/corrections

Bug #5832: correctly process clicker wildcard entries

Replace Scantron(tm) by "bubblesheet"

- Excluding redesign-related changes for now (1.533, 1.537, and 1.538).

Don't translate separator character between and domain.
Advantages: No need to change the translation files, if...
  - different separator character should be used
  - different output is wanted, e.g. just username, plainname, both, etc.
  - different style should be used for output
(work in progress to make this even more flexible by introducing a global function usernamewrapper)

"Student/Employee ID"
- Consistent wording
- Lower-/Uppercase
- German wording: "Studierenden-/Mitarbeiter-ID" (Studierende_n_)
- Removed unused translation phrases
- not just "ID" (i.e. could be "Course ID"; more to do)

Replacing Scantron(tm) with bubblesheet

Make clearer what each failure mode means
Don't generate a one-kilometer-long page when only uploading grades from
one section in a large class.

We are not saving the students themselves.

XHTML conform selected/checked/multiple HTML attributes
and optimized spacing

- Upload scantron data file.
 - Eliminate copy and pasted code from lonnet::userfileupload() by calling the routine directly.
 - Pass scantron as a subdir name (although files still currently stored in top level userfiles directory).

- Bug 3576
 - New routine: &validate_uploaded_scantron_file().
   - Used to compare studentIDs in scantron file with IDs in course roster.
   - Report outcome to uploader.
 - Remove escaped quotes from &mt().
 - Move HTML (<span></span) to [_1], [_2] etc. in &mt().
 - Some wording changes.

- Bug 5914.
  - Selection of "Scantron Operator" role launches Scantron data upload page (from grades.pm)
  - Add header to upload table (until such time as breadcrumbs are updated for grades.pm).

 - Scantron Uploader's upload screen.
   - Replace table tags with pick_box().
   - Link to syllabus adjacent to Course Name textbox (becomes active when a course has been chosen.
   - Javascript alert on submit if no course selected.
   - Domain select box replaced with static domain ($env{'request.role.domain'})

- HTML correction:
  Added missing quotation mark to border attribute value of icon indicating which file type has been uploaded
- XHTML:
  Added missing quotation marks to HTML attribute values

XHTML: Added quotes to HTML attribute values in input fields

- Optimized and localized "Access Denied" output

Optimized screen layout for manual grading options list:
- Now use standard pick box for options
- Moved the "Check For Plagiarism" checkbox to the other options

- Fix typos introduced in backports included in 1.528.2.11.

- Backport part of 1.538, 1.539, 1.540, 1.548, 1.549, 1.550, 1.551, 1.552, 1.553, part of 1.554, 1.555, 1.558, 1.559, 1.560

- Apply &mt() to complete phrases.

- Revert part of rev. 1.539. &mt() not required (Apache::lonlocal::texthash() already in use).

- XHTML: readonly
- Use double quotes for HTML attributes

Efficiency changes.
-  Scantron grading and Standalone verification.
   - Eliminate calls to &scantron_partids_tograde() to get responseIDs for each student if the resource does not contain randomlist.
     - Use partIDs/responseIDs retrieved (once) for current grader.

- Additional hash to cache.
  %analyze_cache_formkeys - additional form items used when record stored in %analyze_cache for a given key.

- Additional arg for &get_analyze().
   $add_to_hash - can be a ref to a HASH of additional form items to include in the ssi call.

- &scantron_partids_tograde() includes 'check_parts_withrandomlist' as additional arg
  - so structuretags::start_randomlist() will add parts to $Apache::lonhomework::analyze{'parts_withrandomlist'} an arrayref if the resource contains a randomlist.

consistent wording:
changed "student number" to "student/employee ID"
changed "student ID" to "student/employee ID"
changed "student and eyployee IDs" to "student/employee IDs"

- Prevent ISE by checking that the item returned by &get_order() really is an array reference.
- Pass $uname and $udom as args in &cleanRecord() call when viewing submission record.
  - parts containing <randomlist></randomlist> might give %analyze missing needed key.
    (structuretags::shuffle() can shuffle blocks differently so
    shown foils info is for different $part.$resp from one needed

- Verification of Scantron grading for problems containing randomlists
  - &scantron_partids_tograde()
    - changed to use &get_analyze() to retrieve partIDs/responseIDs
    - subroutine moved to be within scope of %analyze_cache.
    - needs to be called for each user (randomlist -> possibly different responseIDs for different users).

- Robustness against changes to lonxml::counter being made by multitasking instructors.
  - scantron_questnum_start.$part.$id form element passed in ssi call for scantron grading.
    - where available, this value is used within response::getresponse() instead of lonxml::counter to determine starting column for scantron line.

- Second pass (if optional inbuilt verification enabled during scantron grading).
  - Added missing call to repeat &grade_student_bubbles() when anomaly is detected during first pass.

****
Work-in-progress - possible performance hit - perhaps check if resource includes <randomlist></randomlist> before calling &scantron_partids_tograde() for each user?

Consistent wording "receipt"

- Starting with lonnet.pm rev 1.981, default for &lonnet::delenv() has been to \Q escape arg in regexp used to identify items in environment to delete.
  - No longer need to escape special characters in arg passed to lonnet::delenv().
- Call to lonnet::delenv() in rat/lonuserstate now includes second arg, because in this case first arg is to be treated as a regexp (so \Q escape is not wanted).

Use \Q$part\E in the substitution instead of \w+ because the partID might contain chancters which do not match \w+.

- Bug 5912.
- "Answers" from scantron line only accumulated from columns needed to match total bubble rows.

added quant-calls in the source and edited them in de.pm, because they were not used elsewhere

Localization:
- More eliminations of \' occurrences and mt call optimizations:
  Changed scripts and corresponding translation files
- Script changes related to changes already done in de.pm 1.189 for loncreateuser.pm and grades.pm

- Backport 1.547.

- Typo.

- Backport a part of 1.542 omitted in the original backport.
- Interface to enable the option to peform verification of scantron grading.

- Backport 1.546.

- Eliminate duplicate declaration.

- Backport 1.544.

- Backport 1.545.

- Bug 5792.

- Spelling.

- Some $analysis{} to $analysis->{} changes from backport of 1.542
  (missed on the first pass).

- Backport 1.542, 1.543.

- Remove some duplicate code -- copies already exist
  inside if ($env{'form.verifyrecord'}) {} block.
- Restore Apache::loncommon::connection_aborted() call
  after loop over @resources (removed in 1.542).

- Add option to peform verification of scantron grading.
  - If enabled, scantron bubble record for each student being graded
     will be compared with corresponding last submission stored.
    If the two strings do not match, scantron is re-run for that student.
    The comparison is repeated after the second pass - there is still a mismatch
      an error message is displayed; if they did match a warning is displayed that
      two passes were needed.
  - Code previously in &checkscantron_results() subroutine reused.

- New subroutines created to facilitate code reuse:
&scantron_partids_tograde()
 - returns refs to %analysis and @parts from call to &ssi_with_retries()
 - contains partid.respid for non-hidden parts in a resource.
&grade_student_bubbles()
 - submits %form to &ssi_with_retries() for actual grading
&verify_scantron_grading()
 - extracts the equivalent bubble pattern for a particular resource from
   the last submission for a student.
&letter_to_digits()
 - routine to provide a hash for converting letters A -> J to corresponding numbers 1 - 0.

- Backport 1.541.

- Bug 5850.
  - add $partid.$id for essayresponse, reactionresponse and organicresponse to $Apache::lonhomework::analyze{"parts"} arrayref so they are available in grades::scantron_get_maxbubble() when determing bubble lines for each problem part.

more &mt adjustment

inserted and tidied up &mt

Changes related to LON-CAPA redesign project.
Modified grades.pm to use the new menu generator.

- Backport 1.536.

changed name of a new style to LC_*** and remove div tags because it wasn't
useful at all.

- Line break added before error message for handback file upload errors.
- &mt() added.

- $brcrum is a reference to an anonymous array [] (use $brcrum not @brcrum).
- add sanity checking to confirm that the arg passed was an array ref.
- style
  - indentation
  - exists()

Added Breadcrumbs in Sites that dose not have any
Added a parameter bread_crumbs to the start_page function, so it can handel the creation of breadcrumbs

- Backport 1.532.

Optimized/standarized screen output for first grading screen ("Manual Grading/View Submissions"):
- Replaced special styles with standard fieldsets
- Removed now unused styles from loncommon.pm
- Excluded section preselection from output
These changes also restore the screen layout for the radiobuttons which have moved right to the selection boxes due to new styles.

Replaced all hardcoded nowrap styles with LC_nobreak classes

Moved POD comments to bottom of file

Added/updated/converted POD. Many files had their POD redone to new standards.

Added/modified POD comments

- typo.

- Fix typo.

Localization.
- Missing &mt().

Bug 5695.
Improved fix.  Earlier fix caused other things to break (e.g., scantron grading). Now explicitly pass a grade_noincrement form element in the ssi call to inhibit the unwanted second increment of the counter when displaying submission details ("by datess and submissions" case only).

- style - push(), sort(). keys(), eliminate $_

Ability for a CC to run a comparison of raw scantron data with corresponding submission records for previously graded scantron exams.
- Discrepancies detected.
- Summary of recorded bubbles and corresponding submissions for each student.

Free clicker input field