Unknown Namespace:  #2075

Activity

1. SheetJSDev commented on Aug 1, 2020

   

   SheetJSDev

   on Aug 1, 2020

   Contributor

   More actions

   Can you share a sample file? The check is https://github.com/SheetJS/sheetjs/blob/master/bits/30_ctype.js#L220 which may not even be necessary if Excel doesn't require the namespace for [Content_Types].xml
2. Beful commented on Aug 3, 2020

   

   Beful

   on Aug 3, 2020

   Author

   More actions

   var nContainer = document.getElementById('container')
   var xlsx = new Plus.Xlsx(nContainer)
   xlsx.readFile("/200.xlsx");
3. SheetJSDev commented on Aug 3, 2020

   

   SheetJSDev

   on Aug 3, 2020

   Contributor

   More actions

   Can you share that 200.xlsx file?
4. Beful commented on Aug 3, 2020

   

   Beful

   on Aug 3, 2020

   Author

   More actions

   可以
5. Beful commented on Aug 3, 2020

   

   Beful

   on Aug 3, 2020

   Author

   More actions

   

   又报这个错
6. Beful commented on Aug 3, 2020

   

   Beful

   on Aug 3, 2020

   Author

   More actions

   建筑.xlsx
7. SheetJSDev commented on Aug 3, 2020

   

   SheetJSDev

   on Aug 3, 2020

   Contributor

   More actions

   谢谢!

   Feel free to submit a patch. The change is in bits/22_xmlutils.js:

   --- a/bits/22_xmlutils.js
   +++ b/bits/22_xmlutils.js
   @@ -1,6 +1,6 @@
    var XML_HEADER = '<?xml version="1.0" encoding="UTF-8" standalone="yes"?>\r\n';
    var attregexg=/([^"\s?>\/]+)\s*=\s*((?:")([^"]*)(?:")|(?:')([^']*)(?:')|([^'">\s]+))/g;
   -var tagregex=/<[\/\?]?[a-zA-Z0-9:_-]+(?:\s+[^"\s?>\/]+\s*=\s*(?:"[^"]*"|'[^']*'|[^'">\s=]+))*\s?[\/\?]?>/g;
   +var tagregex=/<[\/\?]?[a-zA-Z0-9:_-]+(?:\s+[^"\s?>\/]+\s*=\s*(?:"[^"]*"|'[^']*'|[^'">\s=]+))*\s*[\/\?]?>/mg;
    
    if(!(XML_HEADER.match(tagregex))) tagregex = /<[^>]*>/g;
    var nsregex=/<\w*:/, nsregex2 = /<(\/?)\w+:/;

   This is a problem with how we are parsing XML tags. There is a CRLF newline sequence within the root tag, just before the closing >:

   00000000: 3c3f 786d 6c20 7665 7273 696f 6e3d 2231  <?xml version="1
   00000010: 2e30 2220 656e 636f 6469 6e67 3d22 5554  .0" encoding="UT
   00000020: 462d 3822 2073 7461 6e64 616c 6f6e 653d  F-8" standalone=
   00000030: 2279 6573 223f 3e0d 3c54 7970 6573 2078  "yes"?>.<Types x
   00000040: 6d6c 6e73 3d22 6874 7470 3a2f 2f73 6368  mlns="http://sch
   00000050: 656d 6173 2e6f 7065 6e78 6d6c 666f 726d  emas.openxmlform
   00000060: 6174 732e 6f72 672f 7061 636b 6167 652f  ats.org/package/
   00000070: 3230 3036 2f63 6f6e 7465 6e74 2d74 7970  2006/content-typ
   00000080: 6573 2220 0d0a 3e0d 3c44 6566 6175 6c74  es" ..>.<Default
                       ::::
   

   The proposed fix changes the RegExp to use the /m flag and changes the penultimate check to match any number of whitespace characters.
8. Beful commented on Aug 3, 2020

   

   Beful

   on Aug 3, 2020

   Author

   More actions

   

   I，m read 虚拟目录 ，请求链接，是不是不支持
9. SheetJSDev

   closed this as completedin abed474on Aug 8, 2020
10. saarCiklum

    added a commit that references this issue on Aug 20, 2020

    whitespace check (fixes SheetJS#2075)

    This issue will close once commit 28212d4 is merged into the 'master' branch.

    28212d4
11. keithpan commented on Dec 15, 2020

    

    keithpan

    on Dec 15, 2020

    Last edited by keithpan

    More actions

    @SheetJSDev
    
    the latest version commit is different from the above reply, it's not changes the penultimate check to match any number of whitespace characters. the latest version is invalid.
    can you help me?
12. SheetJSDev commented on Dec 15, 2020

    

    SheetJSDev

    on Dec 15, 2020

    Contributor

    More actions

    @keithpan does it work if you change the modifier near the end as follows:

    -var tagregex=/<[\/\?]?[a-zA-Z0-9:_-]+(?:\s+[^"\s?>\/]+\s*=\s*(?:"[^"]*"|'[^']*'|[^'">\s=]+))*\s?[\/\?]?>/mg;
    +var tagregex=/<[\/\?]?[a-zA-Z0-9:_-]+(?:\s+[^"\s?>\/]+\s*=\s*(?:"[^"]*"|'[^']*'|[^'">\s=]+))*\s*[\/\?]?>/mg;
                                                                                                    ^

    The caret is marking the modifier that changed. originally it matches up to one whitespace character and the change lets it match any number
13. keithpan commented on Dec 24, 2020

    

    keithpan

    on Dec 24, 2020

    More actions

    @keithpan does it work if you change the modifier near the end as follows:

    -var tagregex=/<[\/\?]?[a-zA-Z0-9:_-]+(?:\s+[^"\s?>\/]+\s*=\s*(?:"[^"]*"|'[^']*'|[^'">\s=]+))*\s?[\/\?]?>/mg;
    +var tagregex=/<[\/\?]?[a-zA-Z0-9:_-]+(?:\s+[^"\s?>\/]+\s*=\s*(?:"[^"]*"|'[^']*'|[^'">\s=]+))*\s*[\/\?]?>/mg;
                                                                                                    ^

    The caret is marking the modifier that changed. originally it matches up to one whitespace character and the change lets it match any number

    @SheetJSDev
    why does the latest version not have this change?
14. ansarikhurshid786 commented on Mar 29, 2022

    

    ansarikhurshid786

    on Mar 29, 2022

    More actions

    I am facing same issue when i publish web server. it is working on local server.