Uploaded image for project: '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
    • 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.

        Gliffy Diagrams

          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: