arelle-release 2.37.46__py3-none-any.whl → 2.38.0__py3-none-any.whl

arelle/WebCache.py

@@ -6,20 +6,29 @@ e.g., User-Agent: Sample Company Name AdminContact@<sample company domain>.com
 
 '''
 from __future__ import annotations
-import contextlib
-
 
-from filelock import FileLock, Timeout
+import calendar
+import contextlib
+import io
+import json
+import logging
+import os
+import posixpath
+import shutil
+import sys
+import time
+import zlib
+from http.client import IncompleteRead
 from pathlib import Path
 from typing import TYPE_CHECKING, Any
-import os, posixpath, sys, time, calendar, io, json, logging, shutil, zlib
-import regex as re
-from urllib.parse import quote, unquote
-from urllib.error import URLError, HTTPError, ContentTooShortError
-from http.client import IncompleteRead
 from urllib import request as proxyhandlers
+from urllib.error import ContentTooShortError, HTTPError, URLError
+from urllib.parse import quote, unquote, urlsplit, urlunsplit
 
 import certifi
+import regex as re
+import truststore
+from filelock import FileLock, Timeout
 
 from arelle.PythonUtil import isLegacyAbs
 
@@ -28,12 +37,6 @@ try:
 except ImportError:
     ssl = None
 
-try:
-    import truststore
-except ImportError:
-    # truststore requires Python > 3.9
-    truststore = None
-
 from arelle.FileSource import SERVER_WEB_CACHE, archiveFilenameParts
 from arelle.PluginManager import pluginClassMethods
 from arelle.UrlUtil import isHttpUrl
@@ -274,8 +277,7 @@ class WebCache:
             self.http_auth_handler = proxyhandlers.HTTPBasicAuthHandler()
             proxyHandlers = [self.proxy_handler, self.proxy_auth_handler, self.http_auth_handler]
         if ssl:
-            # Attempt to load the default CA certificates from the OS using truststore if available, else fallback to OpenSSL's default context.
-            context = truststore.SSLContext(ssl.PROTOCOL_TLS_CLIENT) if truststore else ssl.create_default_context()
+            context = truststore.SSLContext(ssl.PROTOCOL_TLS_CLIENT)
             # Include certifi certificates (Mozilla’s carefully curated
             # collection) for systems with outdated certs.
             context.load_verify_locations(cafile=certifi.where())
@@ -567,14 +569,16 @@ class WebCache:
         :param url:
         :return: `url` with scheme-specific-part quoted except for parameter separators
         """
-        urlScheme, schemeSep, urlSchemeSpecificPart = url.partition("://")
-        urlPath, querySep, query = urlSchemeSpecificPart.partition("?")
-        # RFC 3986: https://www.ietf.org/rfc/rfc3986.txt
-        querySafeChars = ';/?'
-        pathSafeChars = querySafeChars + ':@&=+$,'
-        quotedUrlPath = quote(urlPath, safe=pathSafeChars)
-        quotedQuery = quote(query, safe=querySafeChars)
-        return urlScheme + schemeSep + quotedUrlPath + querySep + quotedQuery
+        parts = urlsplit(url)
+
+        # RFC 3986 safe characters: https://www.ietf.org/rfc/rfc3986.txt
+        pathSafe = "/:@!$&'()*+,;="       # path allows sub-delims, ":" and "@"
+        querySafe = "&=:/?@!$'()*+,;[]"   # query allows pchar + "/" + "?"
+
+        quotedPath = quote(parts.path, safe=pathSafe)
+        quotedQuery = quote(parts.query, safe=querySafe)
+
+        return urlunsplit((parts.scheme, parts.netloc, quotedPath, quotedQuery, parts.fragment))
 
     @staticmethod
     def _getFileTimestamp(path: str) -> float:

arelle/XbrlConst.py

@@ -22,10 +22,13 @@ _tuple = tuple  # type: ignore[type-arg]
 xsd = "http://www.w3.org/2001/XMLSchema"
 qnXsdComplexType = qname("{http://www.w3.org/2001/XMLSchema}xsd:complexType")
 qnXsdDocumentation = qname("{http://www.w3.org/2001/XMLSchema}xsd:documentation")
+qnXsdInclude = qname("{http://www.w3.org/2001/XMLSchema}xsd:include")
 qnXsdImport = qname("{http://www.w3.org/2001/XMLSchema}xsd:import")
 qnXsdSchema = qname("{http://www.w3.org/2001/XMLSchema}xsd:schema")
 qnXsdAppinfo = qname("{http://www.w3.org/2001/XMLSchema}xsd:appinfo")
 qnXsdDefaultType = qname("{http://www.w3.org/2001/XMLSchema}xsd:anyType")
+qnXsdElement = qname("{http://www.w3.org/2001/XMLSchema}xsd:element")
+qnXsdAttribute = qname("{http://www.w3.org/2001/XMLSchema}xsd:attribute")
 xsi = "http://www.w3.org/2001/XMLSchema-instance"
 qnXsiNil = qname(xsi, "xsi:nil")  # need default prefix in qname
 qnXsiType = qname(xsi, "xsi:type")
@@ -127,6 +130,7 @@ qnXlinkArcRole = qname("{http://www.w3.org/1999/xlink}xlink:arcrole")
 qnXlinkFrom = qname("{http://www.w3.org/1999/xlink}xlink:from")
 qnXlinkHref = qname("{http://www.w3.org/1999/xlink}xlink:href")
 qnXlinkLabel = qname("{http://www.w3.org/1999/xlink}xlink:label")
+qnXlinkRole = qname("{http://www.w3.org/1999/xlink}xlink:role")
 qnXlinkTo = qname("{http://www.w3.org/1999/xlink}xlink:to")
 qnXlinkType = qname("{http://www.w3.org/1999/xlink}xlink:type")
 xl = "http://www.xbrl.org/2003/XLink"
@@ -138,6 +142,9 @@ qnXlLocatorType = qname("{http://www.xbrl.org/2003/XLink}xl:locatorType")
 qnXlResourceType = qname("{http://www.xbrl.org/2003/XLink}xl:resourceType")
 qnXlArcType = qname("{http://www.xbrl.org/2003/XLink}xl:arcType")
 xhtml = "http://www.w3.org/1999/xhtml"
+qnXhtmlMeta = qname("{http://www.w3.org/1999/xhtml}meta")
+qnXhtmlImg = qname("{http://www.w3.org/1999/xhtml}img")
+qnXhtmlDel = qname("{http://www.w3.org/1999/xhtml}del")
 ixbrl = "http://www.xbrl.org/2008/inlineXBRL"
 ixbrl11 = "http://www.xbrl.org/2013/inlineXBRL"
 ixbrlAll = {ixbrl, ixbrl11}
@@ -178,6 +185,21 @@ ixAttributes = set(
         "tupleID",
     )
 )
+ixbrlTargetElements = frozenset({
+    qnIXbrlFraction,
+    qnIXbrlNonFraction,
+    qnIXbrlNonNumeric,
+    qnIXbrlResources,
+    qnIXbrlTuple,
+})
+ixbrl11TargetElements = frozenset({
+    qnIXbrl11Fraction,
+    qnIXbrl11NonFraction,
+    qnIXbrl11NonNumeric,
+    qnIXbrl11Resources,
+    qnIXbrl11Tuple,
+})
+ixbrlAllTargetElements = ixbrlTargetElements | ixbrl11TargetElements
 conceptLabel = "http://www.xbrl.org/2003/arcrole/concept-label"
 conceptReference = "http://www.xbrl.org/2003/arcrole/concept-reference"
 footnote = "http://www.xbrl.org/2003/role/footnote"

arelle/XmlUtil.py

@@ -285,9 +285,8 @@ def escapedNode(
         for n,v in sorted(elt.items(), key=lambda item: item[0]):
             if n in uriAttrs:
                 v = resolveHtmlUri(elt, n, v).replace(" ", "%20") # %20 replacement needed for conformance test passing
-            # 2022-09-15: assuming n, v are always strings
-            s.append(' {0}="{1}"'.format(qname(elt, cast(str, n)),
-                cast(str, v).replace("&","&").replace('"', '"')))
+            s.append(' {0}="{1}"'.format(qname(elt, n),
+                v.replace("&","&").replace('"', '"')))
     if not start and empty:
         if selfClosable(elt):
             s.append('/')
@@ -401,9 +400,7 @@ def ancestors(
     ancestorLocalNames: str | tuple[str, ...] | None = None
 ) -> list[ModelObject]:
     if ancestorNamespaceURI is None and ancestorLocalNames is None:
-        return [
-            cast(ModelObject, ancestor) for ancestor in element.iterancestors()
-        ]
+        return list(element.iterancestors())
     ancestors = []
     wildNamespaceURI = not ancestorNamespaceURI or ancestorNamespaceURI == '*'
     treeElt = element.getparent()
@@ -635,13 +632,13 @@ def schemaBaseTypeDerivedFrom(
             return child.get("base")  # str | None
         elif child.tag == "{http://www.w3.org/2001/XMLSchema}union":
             return (child.get("memberTypes") or "").split() + [
-                    schemaBaseTypeDerivedFrom(cast(ModelObject, _child))
+                    schemaBaseTypeDerivedFrom(_child)
                     for _child in child.iterchildren(tag="{http://www.w3.org/2001/XMLSchema}simpleType")]  # list[str | QName | list[This func return] | None]
         elif child.tag in ("{http://www.w3.org/2001/XMLSchema}complexType",
                            "{http://www.w3.org/2001/XMLSchema}simpleType",
                            "{http://www.w3.org/2001/XMLSchema}complexContent",
                            "{http://www.w3.org/2001/XMLSchema}simpleContent"):
-            return schemaBaseTypeDerivedFrom(cast(ModelObject, child))
+            return schemaBaseTypeDerivedFrom(child)
     return None
 
 def schemaFacets(
@@ -846,7 +843,7 @@ def copyIxFootnoteHtml(
         if contAt is not None:
             copyIxFootnoteHtml(contAt, tgtHtml, targetModelDocument, withText=withText, isContinChainElt=True, tgtStack=tgtStack, srcLevel=0)
 
-def addComment(parent: ModelObject, commentText: str) -> None:
+def addComment(parent: etree._Element, commentText: str) -> None:
     comment = str(commentText)
     if '--' in comment: # replace -- with - - (twice, in case more than 3 '-' together)
         comment = comment.replace('--', '- -').replace('--', '- -')
@@ -854,7 +851,7 @@ def addComment(parent: ModelObject, commentText: str) -> None:
     parent.append(child)
 
 def addProcessingInstruction(
-    parent: ModelObject,
+    parent: etree._Element,
     piTarget: str | bytes,
     piText: str,
     insertBeforeChildElements: bool = True,
@@ -909,13 +906,10 @@ def setXmlns(modelDocument: etree._ElementTree | ModelDocument, prefix: str | No
         if root.tag == 'nsmap': # already have an xmlns-extension root element
             newmap = root.nsmap
             newmap[prefix] = namespaceURI
-            # 2022-09-16: for some reason prefix is encouraged to always be a str in lxml-stubs,
-            # but '' for default ns is not accepted by lxml nsmap arg and lxml produces and error
-            # see https://github.com/lxml/lxml-stubs/blob/0a9b6099dd39b298fd0ff897dbcd4fed632d8776/lxml-stubs/etree.pyi#L69
-            newroot = etree.Element('nsmap', nsmap=newmap)  # type: ignore[arg-type]  # above note
+            newroot = etree.Element('nsmap', nsmap=newmap)
             newroot.extend(root)
         else:  # new xmlns-extension root
-            newroot = etree.Element('nsmap', nsmap={prefix: namespaceURI})  # type: ignore[dict-item]  # above note
+            newroot = etree.Element('nsmap', nsmap={prefix: namespaceURI})
             comments = []
             comment = root.getprevious()
             while isinstance(comment, etree._Comment):
@@ -939,9 +933,9 @@ def sortKey(
     if parentElement is not None:
         for childLocalName in childLocalNames if isinstance(childLocalNames,tuple) else (childLocalNames,):
             for child in parentElement.iterdescendants(tag="{{{0}}}{1}".format(childNamespaceUri,childLocalName)):
-                value = text(cast(ModelObject, child))
+                value = text(child)
                 if qnames:
-                    _value = prefixedNameToClarkNotation(cast(ModelObject, child), value)
+                    _value = prefixedNameToClarkNotation(child, value)
                     assert isinstance(_value, str)
                     value = _value
                 if childAttributeName is not None:
@@ -1156,7 +1150,7 @@ def elementTagnamesPath(element: etree._Element | ModelObject | None) -> str:
     # returns clark notation absolute path without element sequences
     tagnamesPath: list[str] = []
     while (element is not None):
-        tagnamesPath.insert(0, element.tag)
+        tagnamesPath.insert(0, cast(str, element.tag))
         element = element.getparent()
     return "/".join(tagnamesPath)
 
@@ -1235,7 +1229,8 @@ def writexml(
             if skipInvalid and getattr(node, "xValid", VALID) == INVALID:
                 return
         else:
-            ns, sep, localName = node.tag.partition('}')
+            nodeTag = cast(str, node.tag)
+            ns, sep, localName = nodeTag.partition('}')
             if sep:
                 ns = ns[1:]
                 prefix = xmlnsprefix(node,ns)
@@ -1258,7 +1253,7 @@ def writexml(
             else:
                 attrs["xmlns"] = ns
         for aTag,aValue in node.items():
-            ns, sep, localName = cast(str, aTag).partition('}')
+            ns, sep, localName = aTag.partition('}')
             if sep:
                 prefix = xmlnsprefix(node,ns[1:])
                 if prefix:
@@ -1267,7 +1262,7 @@ def writexml(
                     prefixedName = localName
             else:
                 prefixedName = ns
-            attrs[prefixedName] = cast(str, aValue)
+            attrs[prefixedName] = aValue
         aSortedNames = sorted(attrs.keys())
 
         # should attribute names be indented on separate lines?

arelle/XmlValidate.py

@@ -268,9 +268,7 @@ def validate(
             presentAttributes = set()
         # validate attributes
         # find missing attributes for default values
-        for attrTag_, attrValue_ in elt.items():
-            attrTag: str = cast(str, attrTag_)
-            attrValue: str = cast(str, attrValue_)
+        for attrTag, attrValue in elt.items():
             qn = qnameClarkName(attrTag)
             #qn = qname(attrTag, noPrefixIsNoNamespace=True)
             baseXsdAttrType = None
@@ -458,9 +456,8 @@ def validateValue(
                     xValue = sValue = value
                     xValid = VALID_ID
                 elif baseXsdType == "anyURI":
-                    if value:  # allow empty strings to be valid anyURIs
-                        if UrlUtil.isValidUriReference(value) is None:
-                            raise ValueError("IETF RFC 2396 4.3 syntax")
+                    if not UrlUtil.isValidUriReference(value):
+                        raise ValueError("IETF RFC 2396 4.3 syntax")
                     # encode PSVI xValue similarly to Xerces and other implementations
                     xValue = anyURI(UrlUtil.anyUriQuoteForPSVI(value))
                     sValue = value
@@ -703,8 +700,8 @@ class lxmlSchemaResolver(etree.Resolver):
                 if xml:
                     return self.resolve_string(xml, context, base_url=_url)
             else: # probably no active modelXbrl yet, such as when loading packages, use url
-                return self.resolve_filename(url, context)  # type: ignore[attr-defined]
-        return self.resolve_empty(context)  # type: ignore[attr-defined]
+                return self.resolve_filename(url, context)
+        return self.resolve_empty(context)
 
 def lxmlResolvingParser(cntlr: Cntlr, modelXbrl: ModelXbrl | None = None) -> etree.XMLParser:
     parser = etree.XMLParser(resolve_entities=False)
@@ -757,7 +754,7 @@ def lxmlSchemaValidate(modelDocument: ModelDocument, extraSchema : str | None =
                 for key, val in docTree.getroot().nsmap.items()
                 if key
             }
-            for e in err.error_log:  # type: ignore[attr-defined]
+            for e in err.error_log:
                 if not any(s in e.message for s in (": The QName value", "is not a valid value of the atomic type 'xs:QName'")):
                     # do newer lxml validations have QName whitespace collapsing issue?
                     userFriendlyElementPath = ''

arelle/_version.py

@@ -1,7 +1,14 @@
 # file generated by setuptools-scm
 # don't change, don't track in version control
 
-__all__ = ["__version__", "__version_tuple__", "version", "version_tuple"]
+__all__ = [
+    "__version__",
+    "__version_tuple__",
+    "version",
+    "version_tuple",
+    "__commit_id__",
+    "commit_id",
+]
 
 TYPE_CHECKING = False
 if TYPE_CHECKING:
@@ -9,13 +16,19 @@ if TYPE_CHECKING:
     from typing import Union
 
     VERSION_TUPLE = Tuple[Union[int, str], ...]
+    COMMIT_ID = Union[str, None]
 else:
     VERSION_TUPLE = object
+    COMMIT_ID = object
 
 version: str
 __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
+commit_id: COMMIT_ID
+__commit_id__: COMMIT_ID
 
-__version__ = version = '2.37.46'
-__version_tuple__ = version_tuple = (2, 37, 46)
+__version__ = version = '2.38.0'
+__version_tuple__ = version_tuple = (2, 38, 0)
+
+__commit_id__ = commit_id = None

arelle/api/Session.py

@@ -8,7 +8,7 @@ from __future__ import annotations
 import logging
 import threading
 from types import TracebackType
-from typing import Any, BinaryIO
+from typing import Any, BinaryIO, TypeVar
 
 from arelle import PackageManager, PluginManager
 from arelle.CntlrCmdLine import CntlrCmdLine, createCntlrAndPreloadPlugins
@@ -18,6 +18,9 @@ from arelle.RuntimeOptions import RuntimeOptions
 
 _session_lock = threading.Lock()
 
+# typing.Self can be used once Python 3.10 support is dropped.
+Self = TypeVar("Self", bound="Session")
+
 
 class Session:
     """
@@ -46,7 +49,7 @@ class Session:
                 "Session objects cannot be shared between threads. Create a new Session instance in each thread."
             )
 
-    def __enter__(self) -> Any:
+    def __enter__(self: Self) -> Self:
         return self
 
     def __exit__(
@@ -110,6 +113,12 @@ class Session:
     ) -> bool:
         """
         Perform a run using the given options.
+        When using a sourceZipStream, the name attribute on it will be checked
+        and, if present, assumed to be the effective name of the zip file the
+        stream represents. When validating '.xbri' files, via a sourceZipStream,
+        it is important to set this name attribute to avoid
+        rpe:documentTypeFileExtensionMismatch errors.
+
         :param options: Options to use for the run.
         :param sourceZipStream: Optional stream to read source data from.
         :param responseZipStream: Options stream to write response data to.

arelle/config/disclosuresystems.xsd

@@ -51,6 +51,7 @@
       <xs:attribute name="deiFilerNameElement" use="optional" type="xs:NCName"/>
       <xs:attribute name="deiNamespacePattern" use="optional"/>
       <xs:attribute name="description" />
+      <xs:attribute name="keepOpen" type="xs:boolean"/>
       <xs:attribute name="identifierSchemePattern" use="optional"/>
       <xs:attribute name="identifierValueName" use="optional"/>
       <xs:attribute name="identifierValuePattern" use="optional"/>
@@ -74,6 +75,7 @@
       <xs:attribute name="standardTaxonomyDatabase" />
       <xs:attribute name="standardTaxonomyUrlPattern" />
       <xs:attribute name="options" />
+      <xs:attribute name="arcroleCyclesAllowed" type="xs:string"/>
     </xs:complexType>
   </xs:element>
 </xs:schema>

arelle/config/rosettaEntitlements.plist

@@ -0,0 +1,8 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+    <key>com.apple.security.cs.allow-unsigned-executable-memory</key>
+    <true/>
+</dict>
+</plist>

arelle/conformance/CSVTestcaseLoader.py

@@ -54,7 +54,7 @@ def loadCsvTestcase(
     if tuple(header) != CSV_TESTCASE_HEADER:
         # CSV doesn't have a recognized testcase header.
         raise CSVTestcaseException(f"CSV file {filepath} doesn't have test case header: {CSV_TESTCASE_HEADER}, first row {header}")
-    testcaseElement = etree.Element("testcase", nsmap=TESTCASE_NAMESPACES_BY_PREFIX)  # type: ignore[arg-type]
+    testcaseElement = etree.Element("testcase", nsmap=TESTCASE_NAMESPACES_BY_PREFIX)
     document = ModelDocument.create(
         modelXbrl,
         ModelDocument.Type.TESTCASE,

arelle/formula/XPathContext.py

@@ -781,11 +781,11 @@ class XPathContext:
                     elif axis.endswith("-sibling"):
                         if isinstance(node, ModelObject):
                             targetNodes = [
-                                sibling  # type: ignore[misc]
+                                sibling
                                 for sibling in node.itersiblings(preceding=axis.startswith("preceding"))
                                 if (
-                                    (not ns or ns == sibling.namespaceURI or ns == "*")  # type: ignore[attr-defined]
-                                    and (localname == sibling.localName or localname == "*")  # type: ignore[attr-defined]
+                                    (not ns or ns == sibling.namespaceURI or ns == "*")
+                                    and (localname == sibling.localName or localname == "*")
                                 )
                             ]
                     elif axis == "preceding":

arelle/logging/formatters/LogFormatter.py

@@ -7,6 +7,8 @@ import logging
 from collections import defaultdict
 from typing import Any
 
+from arelle import UrlUtil
+
 
 class LogFormatter(logging.Formatter):
     def __init__(self, fmt: str | None = None, datefmt: str | None = None) -> None:
@@ -39,7 +41,7 @@ def logRefsFileLines(refs: list[dict[str, Any]]) -> str:
     for ref in refs:
         href = ref.get("href")
         if href:
-            hrefWithoutFakeIxdsPrefix = href.rpartition("_IXDS#?#")[2]
+            hrefWithoutFakeIxdsPrefix = UrlUtil.stripIxdsSurrogatePrefix(href)
             fileLines[hrefWithoutFakeIxdsPrefix.partition("#")[0]].add(ref.get("sourceLine") or 0)
     return ", ".join(file + " " + ', '.join(str(line)
                                             for line in sorted(lines, key=lambda l: l)

arelle/packages/report/ReportPackage.py

@@ -7,7 +7,7 @@ from __future__ import annotations
 import json
 import os
 import zipfile
-from collections import Counter, defaultdict
+from collections import defaultdict
 from dataclasses import dataclass
 from pathlib import Path, PurePosixPath
 from typing import TYPE_CHECKING, Any, cast
@@ -19,13 +19,22 @@ if TYPE_CHECKING:
     from arelle.FileSource import FileSource
 
 
+def pathsContainReportsDirectory(paths: set[str], topLevelDir: str) -> bool:
+    reportDirectory = f"{topLevelDir}/{Const.REPORTS_DIRECTORY}"
+    if reportDirectory in paths:
+        return True
+    reportDirectory = f"{reportDirectory}/"
+    return any(path.startswith(reportDirectory) for path in paths)
+
 def getReportPackageTopLevelDirectory(filesource: FileSource) -> str | None:
     packageEntries = set(filesource.dir or [])
-    potentialTopLevelReportDirs = {
-        topLevelDir
-        for topLevelDir in PackageUtils.getPackageTopLevelDirectories(filesource)
-        if any(f"{topLevelDir}/{path}" in packageEntries for path in Const.REPORT_PACKAGE_PATHS)
-    }
+    potentialTopLevelReportDirs = set()
+    for topLevelDir in PackageUtils.getPackageTopLevelDirectories(filesource):
+        if (
+            f"{topLevelDir}/{Const.REPORT_PACKAGE_FILE}" in packageEntries or
+            pathsContainReportsDirectory(packageEntries, topLevelDir)
+        ):
+            potentialTopLevelReportDirs.add(topLevelDir)
     if len(potentialTopLevelReportDirs) == 1:
         return next(iter(potentialTopLevelReportDirs))
     return None
@@ -98,7 +107,7 @@ def getAllReportEntries(filesource: FileSource, stld: str | None) -> list[Report
     if not isinstance(filesource.fs, zipfile.ZipFile):
         return None
     entries = [f.orig_filename for f in filesource.fs.infolist()]
-    if not any(entry.startswith(f"{stld}/{Const.REPORTS_DIRECTORY}") for entry in entries):
+    if not pathsContainReportsDirectory(set(entries), stld):
         return None
     topReportEntries = []
     entriesBySubDir = defaultdict(list)
@@ -206,15 +215,19 @@ class ReportPackage:
         if reportPackageJsonFile:
             reportPackageJson = getPackageJson(filesource, reportPackageJsonFile)
         reports = getAllReportEntries(filesource, stld)
-        if reportPackageJsonFile is None and reports is None:
+        identifedByInspection = reportPackageJsonFile is not None or reports is not None
+        if reportType == Const.ReportType.UNCONSTRAINED_REPORT_PACKAGE and not identifedByInspection:
+            # A zip file, but not a report package.
             return None
-        reportEntriesBySubDir = Counter(report.dir for report in reports or [] if not report.isTopLevel)
+        if reportType is None:
+            # Archive with unrecognized file extension.
+            if not identifedByInspection:
+                # Not a report package
+                return None
+            # Invalid file extension. Validate as report package and block loading reports.
+            reports = None
         if reports is not None and any(report.isTopLevel for report in reports):
             reports = [report for report in reports if report.isTopLevel]
-        if any(subdirCount > 1 for subdirCount in reportEntriesBySubDir.values()):
-            return None
-        if reportType and reportType.isConstrained and len(reports or []) > 1:
-            return None
         return ReportPackage(
             reportPackageZip=filesource.fs,
             stld=stld,

arelle/packages/report/ReportPackageConst.py

@@ -36,7 +36,6 @@ REPORT_PACKAGE_EXTENSIONS = frozenset(
 
 REPORT_PACKAGE_FILE = f"{META_INF_DIRECTORY}/reportPackage.json"
 REPORTS_DIRECTORY = "reports"
-REPORT_PACKAGE_PATHS = [REPORT_PACKAGE_FILE, f"{REPORTS_DIRECTORY}/"]
 
 XBRL_2_1_REPORT_FILE_EXTENSION = ".xbrl"
 JSON_REPORT_FILE_EXTENSION = ".json"

arelle/plugin/inlineXbrlDocumentSet.py

@@ -140,7 +140,7 @@ from arelle.PluginManager import pluginClassMethods
 from arelle.PrototypeDtsObject import ArcPrototype, LocPrototype
 from arelle.PythonUtil import attrdict, isLegacyAbs
 from arelle.RuntimeOptions import RuntimeOptions
-from arelle.UrlUtil import isHttpUrl
+from arelle.UrlUtil import IXDS_DOC_SEPARATOR, IXDS_SURROGATE, isHttpUrl
 from arelle.utils.EntryPointDetection import filesourceEntrypointFiles
 from arelle.ValidateDuplicateFacts import DeduplicationType
 from arelle.ValidateFilingText import CDATApattern
@@ -156,8 +156,6 @@ from arelle.XmlValidate import NONE, VALID
 from arelle.XmlValidate import validate as xmlValidate
 
 DEFAULT_TARGET = "(default)"
-IXDS_SURROGATE = "_IXDS#?#" # surrogate (fake) file name for inline XBRL doc set (IXDS)
-IXDS_DOC_SEPARATOR = "#?#" # the files of the document set follow the above "surrogate" with these separators
 
 MINIMUM_IXDS_DOC_COUNT = 2 # make this 2 to cause single-documents to be processed without a document set object
 
@@ -248,7 +246,7 @@ def inlineXbrlDocumentSetLoader(modelXbrl, normalizedUri, filepath, isEntry=Fals
         for url in modelXbrl.ixdsDocUrls:
             xml.append("<instance>{}</instance>\n".format(url))
         xml.append("</instances>\n")
-        ixdocset = create(modelXbrl, Type.INLINEXBRLDOCUMENTSET, docsetUrl, isEntry=True, initialXml="".join(xml))
+        ixdocset = create(modelXbrl, Type.INLINEXBRLDOCUMENTSET, docsetUrl, isEntry=True, initialXml="".join(xml), entrypoint=kwargs.get('entrypoint'))
         ixdocset.type = Type.INLINEXBRLDOCUMENTSET
         ixdocset.targetDocumentPreferredFilename = None # possibly no inline docs in this doc set
         for i, elt in enumerate(ixdocset.xmlRootElement.iter(tag="instance")):
@@ -739,7 +737,22 @@ def commandLineOptionExtender(parser, *args, **kwargs):
 def commandLineFilingStart(cntlr, options, filesource, entrypointFiles, *args, **kwargs):
     global skipExpectedInstanceComparison
     skipExpectedInstanceComparison = getattr(options, "skipExpectedInstanceComparison", False)
-    inlineTarget = getattr(options, "inlineTarget", None)
+    filingStart(
+        cntlr,
+        entrypointFiles,
+        getattr(options, "inlineTarget", None),
+    )
+
+def guiFilingStart(cntlr, filesource, entrypointFiles, *args, **kwargs):
+    global skipExpectedInstanceComparison
+    skipExpectedInstanceComparison = False
+    filingStart(
+        cntlr,
+        entrypointFiles,
+        inlineTarget=None
+    )
+
+def filingStart(cntlr, entrypointFiles, inlineTarget):
     if inlineTarget:
         if isinstance(entrypointFiles, dict):
             entrypointFiles = [entrypointFiles]
@@ -1018,6 +1031,7 @@ __pluginInfo__ = {
     'InlineDocumentSet.Discovery': inlineDocsetDiscovery,
     'InlineDocumentSet.Url.Separator': inlineDocsetUrlSeparator,
     'InlineDocumentSet.CreateTargetInstance': createTargetInstance,
+    'CntlrWinMain.Filing.Start': guiFilingStart,
     'CntlrWinMain.Menu.File.Open': fileOpenMenuEntender,
     'CntlrWinMain.Menu.Tools': saveTargetDocumentMenuEntender,
     'CntlrCmdLine.Options': commandLineOptionExtender,

arelle/plugin/validate/DBA/DisclosureSystems.py

@@ -1,12 +1,30 @@
 ARL_2022_PREVIEW = "arl-2022-preview"
 ARL_2024_PREVIEW = "arl-2024-preview"
 ARL_2024_MULTI_TARGET_PREVIEW = "arl-2024-multi-target-preview"
+DKFIN_2024_MULTI_TARGET_PREVIEW = "dkfin-2024-multi-target-preview"
+
+ALL_DISCLOSURE_SYSTEMS = [
+    ARL_2022_PREVIEW,
+    ARL_2024_PREVIEW,
+    ARL_2024_MULTI_TARGET_PREVIEW,
+    DKFIN_2024_MULTI_TARGET_PREVIEW
+]
+
+ARL_DISCLOSURE_SYSTEMS = [
+    ARL_2022_PREVIEW,
+    ARL_2024_PREVIEW,
+    ARL_2024_MULTI_TARGET_PREVIEW
+]
 
 STAND_ALONE_DISCLOSURE_SYSTEMS = [
     ARL_2022_PREVIEW,
     ARL_2024_PREVIEW
 ]
 
-MULTI_TARGET_DISCLOSURE_SYSTEMS = [
+ARL_MULTI_TARGET_DISCLOSURE_SYSTEMS = [
     ARL_2024_MULTI_TARGET_PREVIEW
 ]
+
+DKFIN_MULTI_TARGET_DISCLOSURE_SYSTEMS = [
+    DKFIN_2024_MULTI_TARGET_PREVIEW
+]

arelle/plugin/validate/DBA/PluginValidationDataExtension.py

@@ -5,11 +5,10 @@ from __future__ import annotations
 
 from collections import defaultdict
 from dataclasses import dataclass
-from typing import cast
 
 import regex
 
-from arelle.ModelInstanceObject import ModelFact, ModelContext
+from arelle.ModelInstanceObject import ModelContext, ModelFact
 from arelle.ModelValue import QName
 from arelle.ModelXbrl import ModelXbrl
 from arelle.utils.PluginData import PluginData
@@ -46,7 +45,7 @@ class PluginValidationDataExtension(PluginData):
     disclosureOfEquityQns: frozenset[QName]
     consolidatedMemberQn: QName
     consolidatedSoloDimensionQn: QName
-    cpr_regex: regex.regex.Pattern[str]
+    cpr_regex: regex.Pattern[str]
     dateOfApprovalOfAnnualReportQn: QName
     dateOfExtraordinaryDividendDistributedAfterEndOfReportingPeriod: QName
     dateOfGeneralMeetingQn: QName
@@ -146,7 +145,6 @@ class PluginValidationDataExtension(PluginData):
             return self._reportingPeriodContexts
         contexts = []
         for context in modelXbrl.contexts.values():
-            context = cast(ModelContext, context)
             if context.isInstantPeriod or context.isForeverPeriod:
                 continue  # Reporting period contexts can't be instant/forever contexts
             if len(context.qnameDims) > 0: