ixbrl-viewer 1.4.9__py3-none-any.whl → 1.4.11__py3-none-any.whl

iXBRLViewerPlugin/viewer/src/js/viewer.js

@@ -3,9 +3,8 @@
 import $ from 'jquery'
 import { numberMatchSearch, fullDateMatch } from './number-matcher.js'
 import { TableExport } from './tableExport.js'
-import { escapeRegex, viewerUniqueId } from './util.js'
 import { IXNode } from './ixnode.js';
-import { getIXHiddenLinkStyle, runGenerator } from './util.js';
+import { getIXHiddenLinkStyle, runGenerator, escapeRegex, viewerUniqueId, HIGHLIGHT_COLORS } from './util.js';
 import { DocOrderIndex } from './docOrderIndex.js';
 import { MessageBox } from './messagebox.js';
 
@@ -109,7 +108,7 @@ export class Viewer {
         if (this._reportSet.isMultiDocumentViewer()) {
             $('#ixv .ixds-tabs').show();
             for (const [i, doc] of this._reportSet.reportFiles().entries()) {
-                $('<div class="tab">')
+                $('<div class="tab"></div>')
                     .text(doc.file)
                     .prop('title', doc.file)
                     .data('ix-doc-id', i)
@@ -247,8 +246,13 @@ export class Viewer {
         }
     }
 
-    _findOrCreateWrapperNode(domNode) {
+    _findOrCreateWrapperNode(domNode, inHidden) {
         const v = this;
+
+        if (inHidden) {
+            return $(domNode).addClass("ixbrl-element-hidden");
+        }
+
         /* Is the element the only significant content within a <td> or <th> ? If
          * so, use that as the wrapper element. */
         const tableNode = domNode.closest("td,th");
@@ -308,13 +312,13 @@ export class Viewer {
             const reportIndex = $(iframe).data("report-index");
             $(iframe).contents().find("body *").each((m, node) => {
                 const name = localName(node.nodeName).toUpperCase();
-                if (['NONNUMERIC', 'NONFRACTION', 'FOOTNOTE', 'CONTINUATION'].includes(name)) {
+                if (['NONNUMERIC', 'NONFRACTION', 'FOOTNOTE', 'CONTINUATION'].includes(name) && node.hasAttribute('id')) {
                     const nodeId = viewerUniqueId(reportIndex, node.getAttribute('id'));
                     const continuedAtId = viewerUniqueId(reportIndex, node.getAttribute("continuedAt"));
                     if (continuedAtId !== null) {
                         nextContinuationMap[nodeId] = continuedAtId;
                     }
-                    if (name != 'CONTINUATION' && !node.hasAttribute('target')) {
+                    if (name != 'CONTINUATION') {
                         itemContinuationMap[nodeId] = [];
                     }
                 }
@@ -338,6 +342,18 @@ export class Viewer {
         this.itemContinuationMap = itemContinuationMap;
     }
 
+    _getOrCreateIXNode(vuid, nodes, docIndex, isHidden) {
+        // We may have already created an IXNode for this ID from a -sec-ix-hidden
+        // element 
+        let ixn = this._ixNodeMap[vuid];
+        if (!ixn) {
+            ixn = new IXNode(vuid, nodes, docIndex);
+            this._ixNodeMap[vuid] = ixn;
+            ixn.isHidden = isHidden;
+        }
+        return ixn;
+    }
+
     //
     // Traverse the DOM hierarchy to find IX elements, and build maps and add
     // wrapper nodes and classes.
@@ -375,59 +391,48 @@ export class Viewer {
         const isNonFraction = name === 'NONFRACTION';
         const isFact = isNonNumeric || isNonFraction;
         if (n.nodeType === 1) {
-            // Ignore iXBRL elements that are not in the default target document, as
-            // the viewer builder does not handle these, and the builder does not
-            // ensure that they have ID attributes.
             const vuid = viewerUniqueId(reportIndex, n.getAttribute("id"));
-            if (((isFact || isFootnote) && !n.hasAttribute("target"))
-                || (isContinuation && this.continuationOfMap[vuid] !== undefined)) {
-                let nodes;
-                if (inHidden) {
-                    nodes = $(n);
-                } else {
-                    nodes = this._findOrCreateWrapperNode(n);
-                }
-
-                // For a continuation, store the IX ID(s) of the item(s), not the continuation
-                const headId = isContinuation ? this.continuationOfMap[vuid] : vuid;
-                this._addIdToNode(nodes.first(), headId);
+            if (isFact || isFootnote) {
+                // If @id is not present, it must be for a target document that wasn't processed.
+                if (n.hasAttribute("id")) {
+                    let nodes = this._findOrCreateWrapperNode(n, inHidden);
 
-                // We may have already created an IXNode for this ID from a -sec-ix-hidden
-                // element 
-                let ixn = this._ixNodeMap[vuid];
-                if (!ixn) {
-                    ixn = new IXNode(vuid, nodes, docIndex);
-                    this._ixNodeMap[vuid] = ixn;
-                }
-                if (inHidden) {
-                    ixn.isHidden = true;
-                    nodes.addClass("ixbrl-element-hidden");
-                }
-                if (isContinuation) {
-                    nodes.addClass("ixbrl-continuation");
-                }
-                else {
+                    this._addIdToNode(nodes.first(), vuid);
+                    let ixn = this._getOrCreateIXNode(vuid, nodes, docIndex, inHidden);
                     this._docOrderItemIndex.addItem(vuid, docIndex);
-                }
-                if (isNonFraction) {
-                    nodes.addClass("ixbrl-element-nonfraction");
-                    if (n.hasAttribute('scale')) {
-                        const scale = Number(n.getAttribute('scale'));
-                        // Set scale if the value is a valid number and is not a redundant 0/"ones" scale.
-                        if (!Number.isNaN(scale) && scale !== 0) {
-                            ixn.scale = scale;
+
+                    if (isNonFraction) {
+                        nodes.addClass("ixbrl-element-nonfraction");
+                        if (n.hasAttribute('scale')) {
+                            const scale = Number(n.getAttribute('scale'));
+                            // Set scale if the value is a valid number and is not a redundant 0/"ones" scale.
+                            if (!Number.isNaN(scale) && scale !== 0) {
+                                ixn.scale = scale;
+                            }
                         }
                     }
-                }
-                if (isNonNumeric) {
-                    nodes.addClass("ixbrl-element-nonnumeric");
-                    if (n.hasAttribute('escape') && n.getAttribute('escape').match(/^(true|1)$/)) {
-                        ixn.escaped = true;
+                    if (isNonNumeric) {
+                        nodes.addClass("ixbrl-element-nonnumeric");
+                        if (n.hasAttribute('escape') && n.getAttribute('escape').match(/^(true|1)$/)) {
+                            ixn.escaped = true;
+                        }
+                    }
+                    if (isFootnote) {
+                        nodes.addClass("ixbrl-element-footnote");
+                        ixn.footnote = true;
                     }
                 }
-                if (isFootnote) {
-                    nodes.addClass("ixbrl-element-footnote");
-                    ixn.footnote = true;
+            }
+            else if (isContinuation) {
+                if (n.hasAttribute("id") && this.continuationOfMap[vuid] !== undefined) {
+                    let nodes = this._findOrCreateWrapperNode(n, inHidden);
+
+                    // For a continuation, store the IX ID(s) of the item(s), not the continuation
+                    this._addIdToNode(nodes.first(), this.continuationOfMap[vuid]);
+
+                    this._getOrCreateIXNode(vuid, nodes, docIndex, inHidden);
+
+                    nodes.addClass("ixbrl-continuation");
                 }
             }
             else if(name == 'HIDDEN') {
@@ -733,7 +738,7 @@ export class Viewer {
     highlightAllTags(on, namespaceGroups) {
         const groups = {};
         $.each(namespaceGroups, function (i, ns) {
-            groups[ns] = i;
+            groups[ns] = i % HIGHLIGHT_COLORS;
         });
         const reportSet = this._reportSet;
         const viewer = this;

{ixbrl_viewer-1.4.9.dist-info → ixbrl_viewer-1.4.11.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ixbrl-viewer
-Version: 1.4.9
+Version: 1.4.11
 Summary: The Arelle iXBRL Viewer allows iXBRL reports to be viewed interactively in a web browser.
 Author-email: "arelle.org" <support@arelle.org>
 License: Apache-2.0
@@ -29,12 +29,16 @@ Requires-Python: >=3.8
 Description-Content-Type: text/markdown
 License-File: LICENSE
 License-File: NOTICE
-Requires-Dist: arelle-release ==2.*
 Requires-Dist: lxml ==4.*
-Requires-Dist: pycountry ==22.*
+Requires-Dist: pycountry <24,>=22
+Provides-Extra: arelle
+Requires-Dist: arelle-release ==2.* ; extra == 'arelle'
 Provides-Extra: dev
+Requires-Dist: flake8 ==6.1.0 ; extra == 'dev'
+Requires-Dist: lxml-stubs ==0.4.0 ; extra == 'dev'
+Requires-Dist: mypy ==1.8.0 ; extra == 'dev'
 Requires-Dist: nose2 ==0.14.0 ; extra == 'dev'
-Requires-Dist: typing-extensions ==4.8.0 ; extra == 'dev'
+Requires-Dist: typing-extensions ==4.9.0 ; extra == 'dev'
 
 # Arelle iXBRL Viewer
 

{ixbrl_viewer-1.4.9.dist-info → ixbrl_viewer-1.4.11.dist-info}/RECORD

@@ -1,8 +1,8 @@
 iXBRLViewerPlugin/__init__.py,sha256=WU4immfhGAVbyVRSNhMc-jAqcMHtXNu_8r1v0qYQ9Gw,13974
-iXBRLViewerPlugin/_version.py,sha256=sPZ1R9Ki-yJLDq1LG42Cd8JcFKIsznRRWdx6esE-OfY,160
+iXBRLViewerPlugin/_version.py,sha256=HAxYk2WJE6p-QOIdm7l4LeuuoH2nO0U5RfTze4BjJ4M,162
 iXBRLViewerPlugin/constants.py,sha256=kJ-kjt6rGhwBvk8HNbIiXt31b5qe6Lb6qPOXQF6rH_8,833
 iXBRLViewerPlugin/featureConfig.py,sha256=tOViPYmN32i2uIf_T9HGgbkkv41VoTpTjqhn4w9rTgw,194
-iXBRLViewerPlugin/iXBRLViewer.py,sha256=aHcBcnGBu2X4dhdJmALBehJ9jUi29N5Ne_cuHIMmImk,28138
+iXBRLViewerPlugin/iXBRLViewer.py,sha256=tKdit2-Ts98r4vb07uCzL92RAWatx6xj2AxosH0oyq8,27806
 iXBRLViewerPlugin/stubviewer.html,sha256=hc5BeIR7y3ojGRux_Hy5oqEAHdlQ34kk3lL5eH-B4AU,158
 iXBRLViewerPlugin/ui.py,sha256=G2TsBoBLfpmyYQxB01d_7r8IsMEnemayU__fKvSQeLE,9713
 iXBRLViewerPlugin/xhtmlserialize.py,sha256=XG6HRR6qDclSwbrpywFC0v8Ktb4-YZj-9IV_gqroCOY,6766
@@ -11,17 +11,17 @@ iXBRLViewerPlugin/viewer/version.js,sha256=7b5si8UmaS7QqALQIP-wJ0I8onKFox8VyaAiU
 iXBRLViewerPlugin/viewer/webpack.common.js,sha256=AsiOVAW2XbmSNjv6Kl6LC4VUM2rA9UyQ8vwhz_1jb1Y,1320
 iXBRLViewerPlugin/viewer/webpack.dev.js,sha256=R9AwY_TBrg9otvXpDxT1UsGg5eMqi4aXW7C7EIsHzZk,551
 iXBRLViewerPlugin/viewer/webpack.prod.js,sha256=vfLowWC1EOty0zbq9P-usGaJ3w-JoERpJrYaE9MfoSE,499
-iXBRLViewerPlugin/viewer/dist/ixbrlviewer.js,sha256=3M3tGeDXRyNIgxx7bfu4trbAIu35wcygcDjNhcQVmDE,860807
+iXBRLViewerPlugin/viewer/dist/ixbrlviewer.js,sha256=olTXd-r1RfPVMEeDIYg21KzosMWdgBinU2EfW2uFJ3c,861629
 iXBRLViewerPlugin/viewer/dist/ixbrlviewer.js.LICENSE.txt,sha256=eA5xbvB65RVR5QmweqeHe4sYtW3dzEmgLhUDFOGr9m4,1697
 iXBRLViewerPlugin/viewer/src/html/fact-details.html,sha256=P9BJw1OK4nIYfoT4KPwW1WP94ZeD7v1nisvSKMeSnU8,1495
 iXBRLViewerPlugin/viewer/src/html/footnote-details.html,sha256=9I7y0KM7xsPzbiZheuHkm1k4nHNXD8S9qxvD_Y0I1Us,278
-iXBRLViewerPlugin/viewer/src/html/inspector.html,sha256=7NmtRHQjsF2dVdsijgcKozPc_vwwEiDG0aj7v19mnV4,18109
+iXBRLViewerPlugin/viewer/src/html/inspector.html,sha256=PM-c68WmPVYrz6_F8BnGzU0uT0_RlmL3ZBK8f6kUv5U,18101
 iXBRLViewerPlugin/viewer/src/i18n/en/currencies.json,sha256=p2vXnQcaRWX3tDY8E52AaIeKhbmcNNhPfTBHlL5AL_U,315
 iXBRLViewerPlugin/viewer/src/i18n/en/referenceparts.json,sha256=rp0p7aWt0U2Ny3ai_fadZhdV-LWx-9-tgKWQmggoewM,207
-iXBRLViewerPlugin/viewer/src/i18n/en/translation.json,sha256=cr5nI0IHci9vyRvPB6OECqeYesG49PdCbrgKgczApR8,3819
+iXBRLViewerPlugin/viewer/src/i18n/en/translation.json,sha256=sSVODHTdcZNkAkdOc9x8H55yMVAhWyunFzEhZ8hH4YM,3814
 iXBRLViewerPlugin/viewer/src/i18n/es/currencies.json,sha256=YzzYkYpUAgYFBDROUosaxoVbft5zPXKJ4zI9Zb2nsqQ,171
 iXBRLViewerPlugin/viewer/src/i18n/es/referenceparts.json,sha256=X6l9IS1w9LSItMaS14q9gqDIu1jaeHthuhOogaBLRTI,237
-iXBRLViewerPlugin/viewer/src/i18n/es/translation.json,sha256=H4OgQvgSkfUeI3qMdqr7K46xIIMZLAhroU3z39Ni_sg,3525
+iXBRLViewerPlugin/viewer/src/i18n/es/translation.json,sha256=W7G0nkPF-ajvL3NZP398dFsroSwca-dDBGLPEffm3u8,3522
 iXBRLViewerPlugin/viewer/src/icons/arrow-down.svg,sha256=FjlACizI1NEMLQMyeVSfwsnP1t63HnGBQhmiFLgqDCI,403
 iXBRLViewerPlugin/viewer/src/icons/arrow-up.svg,sha256=-1ZLPAkpGUH4btofeJ_YLyw04dzoWNi_QAm6LK6yYh8,403
 iXBRLViewerPlugin/viewer/src/icons/chevron-down.svg,sha256=V-vUnVI2ixt8EkLYrtNpmW2OT8Iq9u5abEADzi2hlEA,462
@@ -54,25 +54,25 @@ iXBRLViewerPlugin/viewer/src/img/inline-viewer.svg,sha256=7gwas3nHxMunJIhb3clFbS
 iXBRLViewerPlugin/viewer/src/img/powered-by-workiva.png,sha256=4XE5yykXtKUxF2hytq9Uqjx64pa7AgOuidVuB5_04wU,10362
 iXBRLViewerPlugin/viewer/src/img/powered-by-workiva.svg,sha256=Zf2JArzszZCn7zCkHNVZiW8bhlDwPEdgIDJLJe86W30,7620
 iXBRLViewerPlugin/viewer/src/js/accordian.js,sha256=hntVW5QfzHpx_2K1ysbJNDfLR63F_PLTiPg4rEbGs2Y,1842
-iXBRLViewerPlugin/viewer/src/js/aspect.js,sha256=xVN2GdJxxhHJbMq20IDp_2JoLv0O4GWueTBntZrvXDU,2218
-iXBRLViewerPlugin/viewer/src/js/aspect.test.js,sha256=hnht5NazaC4HMoFC7prUEdL1g9oIgkZAFKpL01Z3usg,4242
+iXBRLViewerPlugin/viewer/src/js/aspect.js,sha256=RBJ3-9nT8-iAwhYa7m3g4203lDONfJhbBG40-fkhhp0,2211
+iXBRLViewerPlugin/viewer/src/js/aspect.test.js,sha256=ctDakUsDdJ65uuW70WcnsRW7keVBOsxWrKqD6PZsPC8,4269
 iXBRLViewerPlugin/viewer/src/js/calculations.js,sha256=xPT2e8CtDXgnJA5DnAWouyeYJijNSQBrMb1D6iUM0KI,3388
-iXBRLViewerPlugin/viewer/src/js/chart.js,sha256=w0JS5TrcJlxlMpoT7v4Qjw73seLPbr1hPNu9MFuhwgI,7000
-iXBRLViewerPlugin/viewer/src/js/chart.test.js,sha256=GepXX4pG2se2DAXr99JEKQJV955Lws8gyPo-MXK8Jtk,3386
+iXBRLViewerPlugin/viewer/src/js/chart.js,sha256=B-Wdr4Rshdr5LWknJyv8u9Ro_8GYQL3N-ZAiotFkcps,6997
+iXBRLViewerPlugin/viewer/src/js/chart.test.js,sha256=LWH7Qb-ABufPGxyAomX6q7PJgMpUWNvHskHLYCiepiE,3379
 iXBRLViewerPlugin/viewer/src/js/concept.js,sha256=LCtYnEqmVWw2d61SF6an80RhO78swfxNnuRnhtSz7tM,1176
-iXBRLViewerPlugin/viewer/src/js/concept.test.js,sha256=g3fIk4g0twrwhyoOX7eGX2HuMRzN_MACi47WPQ15pAk,2558
+iXBRLViewerPlugin/viewer/src/js/concept.test.js,sha256=ZQHRJSxrDo4iB9srYxdmpLtgqFP1iWiCIKrezLIgthw,2543
 iXBRLViewerPlugin/viewer/src/js/dialog.js,sha256=09uGcyAcaWLyMyUNDmP9XqYKsuDiAAEIQUM6YLwG_j4,1058
 iXBRLViewerPlugin/viewer/src/js/docOrderIndex.js,sha256=rhNooqCDRU0f8Sb-adHnwqF7kq6vJA_rskGh6oPqdx8,674
-iXBRLViewerPlugin/viewer/src/js/fact.js,sha256=jYnI15b8qthBWJfquZ5fvJwx7fNt8jV5lEGwGR7fV9g,10103
-iXBRLViewerPlugin/viewer/src/js/fact.test.js,sha256=91RJpP2gFvcbI8YRL9YgtTDvK81xmNSDPGK1Vte6ERE,22957
+iXBRLViewerPlugin/viewer/src/js/fact.js,sha256=PHUSXU2w-wlQLypyad358OPenl1DS6-Cs0OEGzZ9D1Y,9693
+iXBRLViewerPlugin/viewer/src/js/fact.test.js,sha256=-DVTQ1rr3uNwPCILZFE9iOUHtv6m86MIiBj0EyRgvWo,22681
 iXBRLViewerPlugin/viewer/src/js/factset.js,sha256=qhMoLwAB5T6GtWyaBXdLnDfGrkEG9a6s6OeygO-6T7M,3861
-iXBRLViewerPlugin/viewer/src/js/factset.test.js,sha256=DFPx4rLHk9HQQTpwVoehSjJEgyU2gBdeMsTKqvRNbuQ,6749
+iXBRLViewerPlugin/viewer/src/js/factset.test.js,sha256=Edji12qbFqvkHduK_BkE0wmaycdaG2sQSdXwdCpriBg,6675
 iXBRLViewerPlugin/viewer/src/js/footnote.js,sha256=jZreicIBPAtBjbpoRbLnkSs8fHdCYVlr-mqn_x5fUfw,597
 iXBRLViewerPlugin/viewer/src/js/identifiers.js,sha256=gcGitF1CfiyiUvzWDG6INnFDIBi5jXdBgiqCB8Ip0YE,2260
 iXBRLViewerPlugin/viewer/src/js/identifiers.test.js,sha256=eu2TkHVpuRItIyNdNAuoITnpGFwwAicqMynf0dFreRk,2112
 iXBRLViewerPlugin/viewer/src/js/index.js,sha256=SdxunWx8kNw0-vwwnr58-ac1-eQIE1__Ll7ekNNK79E,232
-iXBRLViewerPlugin/viewer/src/js/inspector.js,sha256=6xI_Qj-Qaqp_-5szctUUeyIxkQCGMfz4mQbCeUWXwLs,45281
-iXBRLViewerPlugin/viewer/src/js/inspector.test.js,sha256=aLmCtcQYtXM4y3EC1LAfwbQD4QDgVYKeZ7SBlAOgzZk,10118
+iXBRLViewerPlugin/viewer/src/js/inspector.js,sha256=CcBbXBcR8vOwbttIMIHoYmt3NOtyJliMCuNGwFhRx5M,45317
+iXBRLViewerPlugin/viewer/src/js/inspector.test.js,sha256=AQjbwTllw_dZfovCfjfHWtt0kdjzDnoXw-mQ0IrHKWk,10086
 iXBRLViewerPlugin/viewer/src/js/interact.min.js,sha256=Yk0ZkCU3IZ0heGijAgUiNVkZ1Djq-MjvbS32DBSrdIs,85048
 iXBRLViewerPlugin/viewer/src/js/ixbrlviewer.js,sha256=3bQpJ_fUJzSQNjFizz6_7jJ5bKctwQq8Bb1sJB-HE2Y,13211
 iXBRLViewerPlugin/viewer/src/js/ixbrlviewer.test.js,sha256=ey61xTRZkP0Kf3W8DVQtZE-PeXwK4OOx51iLySMSBsY,2712
@@ -84,14 +84,14 @@ iXBRLViewerPlugin/viewer/src/js/number-matcher-preprocess.test.js,sha256=0DaUAwI
 iXBRLViewerPlugin/viewer/src/js/number-matcher.js,sha256=mGqt20dBAHmWN9a96qNzC3lrJoB-QSh0_hf1Ihf4Emg,6133
 iXBRLViewerPlugin/viewer/src/js/number-matcher.test.js,sha256=mNgnB46-p7NxWa6aQvI83GRDupxncvGeDb59KuI8CMM,4059
 iXBRLViewerPlugin/viewer/src/js/outline.js,sha256=pf5l0d8Cplu-uMBJSOmSfyx4AieNgm3C-GAnfxBHAZY,6594
-iXBRLViewerPlugin/viewer/src/js/outline.test.js,sha256=GJtAB_1Yj-sxoiNQJOZUjSuVJ7TE4NHRW-259HBllrA,13677
+iXBRLViewerPlugin/viewer/src/js/outline.test.js,sha256=PaUzfy675G04YcjY8CcFDLCWi87Xx2hjqh1pogdetYY,13661
 iXBRLViewerPlugin/viewer/src/js/period.js,sha256=D9zXSOr19trWCmHizgCe5DQ612HB6IlAWmI5Rh4I8ZM,2267
 iXBRLViewerPlugin/viewer/src/js/period.test.js,sha256=nMamproI2VukUnO0wm3SKQA_rnjwNkwkzyn4uhke0kc,5417
 iXBRLViewerPlugin/viewer/src/js/qname.js,sha256=6MqMBbgziuPPNHKS3UUGyYIgHWygaAIf0mcpK95hwT8,282
-iXBRLViewerPlugin/viewer/src/js/report.js,sha256=6A9it4-1mIIGBgwDdVLRG0BUyS8OljodHuXhBqmchF8,8567
-iXBRLViewerPlugin/viewer/src/js/report.test.js,sha256=tedkrEUoUrFv9_zfU3ekJT-HQNEOVH8PftxgJof0saQ,5041
+iXBRLViewerPlugin/viewer/src/js/report.js,sha256=4vP4LMHmzAFWQMKKbRQ-9z16PgZvVM1rkztvyNqiBeg,8694
+iXBRLViewerPlugin/viewer/src/js/report.test.js,sha256=OpWSRo0oWQY2nwwD2u1L_y8APB9kjkJ4pV89WuTmVzo,5068
 iXBRLViewerPlugin/viewer/src/js/reportset.js,sha256=4_f6zphwC2firfYeiX4gd4DyyvZ-MFMaq1Idej3nFK8,7832
-iXBRLViewerPlugin/viewer/src/js/reportset.test.js,sha256=ZSXsojqB_aWpNq5XUoMlfKG5KsUtZTb7UtaGX1_-Hrs,12573
+iXBRLViewerPlugin/viewer/src/js/reportset.test.js,sha256=qzXGfqKlEu3pGk4m_iMwFdXsIxTCoRM7f4bUrYWIsUU,12558
 iXBRLViewerPlugin/viewer/src/js/search.js,sha256=M7rZ7IVqzUbsLusgbu2ohhLnaRRigAVYJ2Bb8jmjKPQ,5490
 iXBRLViewerPlugin/viewer/src/js/search.test.js,sha256=YGUrgxKjNC0CoY9CJm5FbpBi5tepqMcGriIv7fw19aQ,20925
 iXBRLViewerPlugin/viewer/src/js/summary.js,sha256=86-BeF_h6PxH4ztY2nzBX04yvdvDN1s539S1k9PahZ4,4437
@@ -99,12 +99,12 @@ iXBRLViewerPlugin/viewer/src/js/summary.test.js,sha256=-fMLTuFz0JTk6dauAeB_QBekP
 iXBRLViewerPlugin/viewer/src/js/tableExport.js,sha256=kE0Zyr7MPzcopcc0yy-NKiRwr2ZpyvQx87PYBrosIgc,9669
 iXBRLViewerPlugin/viewer/src/js/test-utils.js,sha256=xPjpwZxuxXcCNqDnRacN46ldYP9gL_a35R8nhvTkXRk,1034
 iXBRLViewerPlugin/viewer/src/js/textblockviewer.js,sha256=lCAZeN_wrSwmYie_rwdeIFCIjiOSblT0K9Vslqwktc8,1874
-iXBRLViewerPlugin/viewer/src/js/unit.js,sha256=KBaFtwxdGU0a9aBDVzDB6FEcD8mfIaIUDLLBh5YRJR4,2192
-iXBRLViewerPlugin/viewer/src/js/unit.test.js,sha256=D11c0Z0j5Xpci3FgiCCDjqLKtIFWVdTYJ9R5AvYAgx0,3084
-iXBRLViewerPlugin/viewer/src/js/util.js,sha256=AlX2qgDGfj5mc9XDpL64qnXxNW7qEe_97ILhCsTRPhw,6104
+iXBRLViewerPlugin/viewer/src/js/unit.js,sha256=eHaCTE3snK7uuXK48VK1yPPpl0MAQReqPGWKhBvzwPQ,1638
+iXBRLViewerPlugin/viewer/src/js/unit.test.js,sha256=l4nELK-gK2uJG6jXHEXwnAKzCCCQ7V_oTySaUN_JnhQ,2801
+iXBRLViewerPlugin/viewer/src/js/util.js,sha256=oF6wGOTODyMNrQ9N91FqOYamW_bLmLYUdiYaGnviH-c,6793
 iXBRLViewerPlugin/viewer/src/js/util.test.js,sha256=9z1y2ImH2t8GKAF5wpkXBADkeQd6X6H7JBejbaSHRwg,6460
 iXBRLViewerPlugin/viewer/src/js/validationreport.js,sha256=zWA9_8sWavX2Fi8fwEyDPABxWL_XSzvNQ21O1Qu2ayY,644
-iXBRLViewerPlugin/viewer/src/js/viewer.js,sha256=Uso_MieWmpbAhtVtDUmsk11ktIjjrewTryL4qSwETu0,34884
+iXBRLViewerPlugin/viewer/src/js/viewer.js,sha256=9ob_js01hDd2B8IygYhTo-GsUPUsG84_Jx_co-NEasM,35018
 iXBRLViewerPlugin/viewer/src/js/viewerOptions.js,sha256=Cvj-gAhqNbdyoUAhUKt-EoTKVOcBR5OjG6y-11hF_hY,164
 iXBRLViewerPlugin/viewer/src/less/accordian.less,sha256=MMYLCpZprb0Y8Mc8bkTL0qQgIsgLbk3u08ZBaCnYXnY,582
 iXBRLViewerPlugin/viewer/src/less/block-list.less,sha256=p2ERp2ahRGSr3Mkj9KhB0eZBzUPEFnhv9Y_58PlWuxs,502
@@ -146,12 +146,12 @@ tests/puppeteer/tools/generate.sh,sha256=A-SUACwIh_OLlV6nPh2HH21grYhPLIqwm0PpygR
 tests/unit_tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/unit_tests/iXBRLViewerPlugin/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/unit_tests/iXBRLViewerPlugin/mock_arelle.py,sha256=Dm11pBwEqDmcCc4mkxNciFEEM_DbgtzQ2QYNP1y_ImE,1307
-tests/unit_tests/iXBRLViewerPlugin/test_iXBRLViewer.py,sha256=arjJXU5jWkh_JmEuZIGXVJ0Ory6_uGUvPcQLM9o6TZA,31583
+tests/unit_tests/iXBRLViewerPlugin/test_iXBRLViewer.py,sha256=naSIovmZmn53p8u7csJ7lAl0W8_XbMUIFcCQi-IHrjQ,32024
 tests/unit_tests/iXBRLViewerPlugin/test_xhtmlserialize.py,sha256=BQNN1sC899sZzV1RN3wuFj9uwi8ioaRJcONxTiTdfZM,12570
-ixbrl_viewer-1.4.9.dist-info/LICENSE,sha256=TZJhu77S_2-WQcPAkuIAlQiuuiNqVcuHBjd7z3Y5S08,16645
-ixbrl_viewer-1.4.9.dist-info/METADATA,sha256=YpgLF08ZyS3Oy1M6gl8hvFDpz51X5v10N7WSUSFAsvM,14496
-ixbrl_viewer-1.4.9.dist-info/NOTICE,sha256=-SHDY0OY7s4gm4Rdk5AB3nbnUsrdHEHPdJuGFR_vuM4,566
-ixbrl_viewer-1.4.9.dist-info/WHEEL,sha256=oiQVh_5PnQM0E3gPdiz09WCNmwiHDMaGer_elqB3coM,92
-ixbrl_viewer-1.4.9.dist-info/entry_points.txt,sha256=2XUzP20WGwxdvnugdBybUBwAB3xf_zvrOR5W_smFz_4,65
-ixbrl_viewer-1.4.9.dist-info/top_level.txt,sha256=h8MkrMhC_t2-KbfS1oxJVnFAbn5NZJH8END_BL40mv8,24
-ixbrl_viewer-1.4.9.dist-info/RECORD,,
+ixbrl_viewer-1.4.11.dist-info/LICENSE,sha256=TZJhu77S_2-WQcPAkuIAlQiuuiNqVcuHBjd7z3Y5S08,16645
+ixbrl_viewer-1.4.11.dist-info/METADATA,sha256=fzkmp8hYOp3bHXFUMU4uwbAliyncUKWLXua44Nq8wnY,14685
+ixbrl_viewer-1.4.11.dist-info/NOTICE,sha256=-SHDY0OY7s4gm4Rdk5AB3nbnUsrdHEHPdJuGFR_vuM4,566
+ixbrl_viewer-1.4.11.dist-info/WHEEL,sha256=oiQVh_5PnQM0E3gPdiz09WCNmwiHDMaGer_elqB3coM,92
+ixbrl_viewer-1.4.11.dist-info/entry_points.txt,sha256=2XUzP20WGwxdvnugdBybUBwAB3xf_zvrOR5W_smFz_4,65
+ixbrl_viewer-1.4.11.dist-info/top_level.txt,sha256=h8MkrMhC_t2-KbfS1oxJVnFAbn5NZJH8END_BL40mv8,24
+ixbrl_viewer-1.4.11.dist-info/RECORD,,

tests/unit_tests/iXBRLViewerPlugin/test_iXBRLViewer.py

@@ -14,7 +14,7 @@ from .mock_arelle import mock_arelle
 
 mock_arelle()
 
-from iXBRLViewerPlugin.iXBRLViewer import NamespaceMap, IXBRLViewerBuilder
+from iXBRLViewerPlugin.iXBRLViewer import NamespaceMap, IXBRLViewerBuilder, iXBRLViewerFile
 
 class TestNamespaceMap(unittest.TestCase):
 
@@ -449,7 +449,8 @@ class TestIXBRLViewer(unittest.TestCase):
     @patch('arelle.XbrlConst.conceptReference', 'http://www.xbrl.org/2003/arcrole/concept-reference')
     def test_addConcept_simple_case(self):
         builder = IXBRLViewerBuilder([self.modelXbrl_1])
-        builder.addReport()
+        builder.currentTargetReport = builder.newTargetReport()
+        builder.addSourceReport()["targetReports"].append(builder.currentTargetReport)
         builder.addConcept(self.modelXbrl_1, self.cash_concept)
         self.assertTrue(builder.taxonomyData["sourceReports"][0]["targetReports"][0].get('concepts').get('us-gaap:Cash'))
 
@@ -465,7 +466,7 @@ class TestIXBRLViewer(unittest.TestCase):
     @patch('arelle.XbrlConst.summationItem', 'http://www.xbrl.org/2003/arcrole/summation-item')
     def test_getRelationships_returns_a_rel(self):
         builder = IXBRLViewerBuilder([self.modelXbrl_1])
-        builder.addReport()
+        builder.currentTargetReport = builder.newTargetReport()
         result = builder.getRelationships(self.modelXbrl_1)
         roleMap = builder.roleMap
         siPrefix = roleMap.getPrefix('http://www.xbrl.org/2003/arcrole/summation-item')
@@ -477,10 +478,10 @@ class TestIXBRLViewer(unittest.TestCase):
         """
         elr = "http://example.com/unknownELR"
         builder = IXBRLViewerBuilder([self.modelXbrl_1])
-        builder.addReport()
+        builder.currentTargetReport = builder.newTargetReport()
         builder.addELR(self.modelXbrl_1, elr)
         elrPrefix = builder.roleMap.getPrefix(elr)
-        self.assertEqual(builder.currentReportData.get('roleDefs').get(elrPrefix), None)
+        self.assertEqual(builder.currentTargetReport.get('roleDefs').get(elrPrefix), None)
 
     def test_addELR_with_definition(self):
         """
@@ -488,10 +489,10 @@ class TestIXBRLViewer(unittest.TestCase):
         """
         elr = "ELR"
         builder = IXBRLViewerBuilder([self.modelXbrl_1])
-        builder.addReport()
+        builder.currentTargetReport = builder.newTargetReport()
         builder.addELR(self.modelXbrl_1, elr)
         elrPrefix = builder.roleMap.getPrefix(elr)
-        self.assertEqual(builder.currentReportData.get('roleDefs').get(elrPrefix).get("en"), "ELR Label")
+        self.assertEqual(builder.currentTargetReport.get('roleDefs').get(elrPrefix).get("en"), "ELR Label")
 
     @patch('arelle.XbrlConst.conceptLabel', 'http://www.xbrl.org/2003/arcrole/concept-label')
     @patch('arelle.XbrlConst.conceptReference', 'http://www.xbrl.org/2003/arcrole/concept-reference')
@@ -560,9 +561,7 @@ class TestIXBRLViewer(unittest.TestCase):
             'local-unrecognized-linkbase.xml': ['unrecognizedLinkbase'],
         })
 
-        # There's not actually a good reason for not including "docSetFiles"
-        # now, but it's what we do currently.
-        self.assertNotIn("docSetFiles", jsdata["sourceReports"][0]);
+        self.assertEqual(jsdata["sourceReports"][0]["docSetFiles"], ["a.html"]);
 
     @patch('arelle.XbrlConst.conceptLabel', 'http://www.xbrl.org/2003/arcrole/concept-label')
     @patch('arelle.XbrlConst.conceptReference', 'http://www.xbrl.org/2003/arcrole/concept-reference')
@@ -710,9 +709,15 @@ class TestIXBRLViewer(unittest.TestCase):
             xml = etree.parse(StringIO(xmls))
 
             js_uri = 'https://example.com/script-url'
-            result = self.builder_1.addViewerToXMLDocument(xml, js_uri)
+            viewer_file = iXBRLViewerFile("test.xhtml", xml)
+            result = self.builder_1.addViewerData(viewer_file, js_uri)
 
-            body = xml.getroot()[0]
+            # addViewerData takes a copy, so original body tag should be empty
+            original_body = xml.getroot()[0]
+            self.assertEqual(len(original_body), 0)
+
+            body = viewer_file.xmlDocument.getroot()[0]
+            self.assertEqual(len(body), 4)
             self.assertEqual(body[0].text, 'BEGIN IXBRL VIEWER EXTENSIONS')
             self.assertEqual(body[1].tag, '{http://www.w3.org/1999/xhtml}script')
             self.assertEqual(body[1].prefix, None)