Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-22073

RIS files starting with Byte Order Mark (BOM) fail to import the first citation.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: CLOSED
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8.1
    • Fix Version/s: 10.0
    • Component/s: Citations
    • Labels:
      None
    • Test Plan:
      Hide

      Download the two files attached to the JIRA ticket (Paleo6a-bom.txt and Paleo6a-nobom.txt).

      Use hexdump to confirm that Paleo6a-bom.txt starts with a BOM and that Paleo6a-nobom.txt does not (as described in the ticket).

      Create two citation lists. Upload Paleo6a-bom.txt to one citation list and paleo6a-nobom.txt to the other. The result should be two identical citation lists with three citations each.

      Show
      Download the two files attached to the JIRA ticket (Paleo6a-bom.txt and Paleo6a-nobom.txt). Use hexdump to confirm that Paleo6a-bom.txt starts with a BOM and that Paleo6a-nobom.txt does not (as described in the ticket). Create two citation lists. Upload Paleo6a-bom.txt to one citation list and paleo6a-nobom.txt to the other. The result should be two identical citation lists with three citations each.

      Description

      A text file which starts with a Byte Order Mark (BOM) doesn't import correctly, the first citation in the list fails to import. http://en.wikipedia.org/wiki/Byte_order_mark As mentioned in the Wikipedia article Notepad on Windows will add a BOM to UTF-8 files.

      Attached is the same list twice, once with a UTF-8 BOM (EF BB BF) and one without. You can see the BOM in a hexdump of the file:

      $ hexdump -n 16 Paleo6a-bom.txt
      0000000 ef bb bf 54 59 20 20 2d 20 4a 4f 55 52 0d 0a 41
      0000010

        Gliffy Diagrams

          Zeplin

            Attachments

            1. Paleo6a-bom.txt
              2 kB
            2. Paleo6a-nobom.txt
              2 kB
            3. SAK-22073.patch
              3 kB

              Issue Links

                Activity

                  People

                  Assignee:
                  a.fish@lancaster.ac.uk Adrian Fish
                  Reporter:
                  buckett Matthew Buckett
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration