Contrib: Gradebook2
  1. Contrib: Gradebook2
  2. GRBK-698

gb2-import structure - import gives blank, grey screen if there is a problem w/imported spreadsheet

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.0-b9
    • Fix Version/s: 1.3.0, Sprint, 1.8.0-a7, 1.8.0
    • Component/s: Services
    • Labels:
      None
    • Target Version:
    • QA Status:
      PASS
    • Previous Issue Keys:

      Description

      rSmart dev server
      SmartSite@UCDavis - 2.6.4.0 - Sakai 2.6.2 - Server "ucddev1"

      initial problem of inability to import structure of gb was user error.
      I'd renamed several items and didn't think to rename the corresponding Comments : columns.
      Mike figured it out.

      BUT this JIRA issue was precipitated by doing the above.
      upon attempt to import the spreadsheet, after the "reading file" popup, the
      screen clears and you're left with a blank, gray screen with "Return"
      at the bottom. if you "Return" you're back in the initial gb2 screen with
      nothing imported.

      Mike says this is an issue as there is no user message/feedback that the user has
      done something wrong with the file and correction/s need to be made.

      so the bug is to fix the code so the user gets an error message of some sort.

        Issue Links

          Activity

          Hide
          Jon Gorrono added a comment -
          This appears to have been fixed just prior to the creation of this ticket... error notes are added to the return data by the commit James did (67092) on 04/11/2010 and this was created a little over a month later... I suspect that a release had not been done and this was based on prod...The next release was in Aug.

          At any rate... the specific condition cited that led to the creation of the ticket does not reproduce the same error.

          The English grammar is pretty second-rate, but there is a message
          Show
          Jon Gorrono added a comment - This appears to have been fixed just prior to the creation of this ticket... error notes are added to the return data by the commit James did (67092) on 04/11/2010 and this was created a little over a month later... I suspect that a release had not been done and this was based on prod...The next release was in Aug. At any rate... the specific condition cited that led to the creation of the ticket does not reproduce the same error. The English grammar is pretty second-rate, but there is a message
          Hide
          Jon Gorrono added a comment -
          I've re-added 1.8.0 to the fix version so that it can be tracked with documentation that targeted that version
          Show
          Jon Gorrono added a comment - I've re-added 1.8.0 to the fix version so that it can be tracked with documentation that targeted that version
          Hide
          Jon Gorrono added a comment -
          have to handle clickers saved as xls... reopening
          Show
          Jon Gorrono added a comment - have to handle clickers saved as xls... reopening
          Hide
          Jon Gorrono added a comment -
          - jexlcel searches are case-sensitive, updated injected column names to reflect proper case
            - some i18n
             - set the imported file 'scantron' flag if it is a clicker to overload scantron code
          Show
          Jon Gorrono added a comment - - jexlcel searches are case-sensitive, updated injected column names to reflect proper case   - some i18n    - set the imported file 'scantron' flag if it is a clicker to overload scantron code
          Hide
          Peter Peterson added a comment -
          So Jon, is this confirmed as already fixed, if so can you close this issue please?
          Show
          Peter Peterson added a comment - So Jon, is this confirmed as already fixed, if so can you close this issue please?
          Hide
          Jon Gorrono added a comment - - edited
          I don't see any QA results... did anybody test this?

          My first comment on this ticket by itself is a little misleading, I ended up making some code changes...so, someone should test clickers that are saved as both CSV and XLS
          Show
          Jon Gorrono added a comment - - edited I don't see any QA results... did anybody test this? My first comment on this ticket by itself is a little misleading, I ended up making some code changes...so, someone should test clickers that are saved as both CSV and XLS
          Hide
          Peter Peterson added a comment -
          Tested several clicker files and also scantron files. Importing to GB. Errors were processed normally and could not recreate the described problem.

          QA PASS
          Show
          Peter Peterson added a comment - Tested several clicker files and also scantron files. Importing to GB. Errors were processed normally and could not recreate the described problem. QA PASS
          Hide
          Peter Peterson added a comment -
          Closing issue
          Show
          Peter Peterson added a comment - Closing issue

            People

            • Assignee:
              Unassigned
              Reporter:
              nancy wilkison
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: