arelle-release 2.37.49__tar.gz → 2.37.50__tar.gz

{arelle_release-2.37.49 → arelle_release-2.37.50}/CONTRIBUTING.md

@@ -68,12 +68,12 @@ Here's how to set up your environment:
   2. Install a supported version of Python.
     For example, 
     
-    pyenv install 3.13.6
+    pyenv install 3.13.7
 
   3. Create a virtual env using the Python version you just installed.
     For example, 
 
-    PYENV_VERSION=3.13.6 pyenv exec python -m venv venv
+    PYENV_VERSION=3.13.7 pyenv exec python -m venv venv
   4. Activate your environment: 
     
     source venv/bin/activate

{arelle_release-2.37.49 → arelle_release-2.37.50}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arelle-release
-Version: 2.37.49
+Version: 2.37.50
 Summary: An open source XBRL platform.
 Author-email: "arelle.org" <support@arelle.org>
 License-Expression: Apache-2.0
@@ -31,36 +31,38 @@ License-File: LICENSE.md
 Requires-Dist: bottle<0.14,>=0.13
 Requires-Dist: certifi
 Requires-Dist: filelock
-Requires-Dist: jsonschema==4.*
-Requires-Dist: isodate==0.*
-Requires-Dist: lxml<6,>=4
+Requires-Dist: isodate<1,>=0
+Requires-Dist: jsonschema<5,>=4
+Requires-Dist: lxml!=6.0.0,<7,>=4
 Requires-Dist: numpy<3,>=1
-Requires-Dist: openpyxl==3.*
+Requires-Dist: openpyxl<4,>=3
 Requires-Dist: pillow<12,>=10
-Requires-Dist: pyparsing==3.*
-Requires-Dist: python-dateutil==2.*
+Requires-Dist: pyparsing<4,>=3
+Requires-Dist: python-dateutil<3,>=2
 Requires-Dist: regex
-Requires-Dist: truststore==0.*; python_version > "3.9"
-Requires-Dist: typing-extensions==4.*
+Requires-Dist: truststore<1,>=0; python_version > "3.9"
+Requires-Dist: typing-extensions<5,>=4
 Provides-Extra: crypto
-Requires-Dist: pycryptodome==3.*; extra == "crypto"
+Requires-Dist: pycryptodome<4,>=3; extra == "crypto"
 Provides-Extra: db
-Requires-Dist: pg8000==1.*; extra == "db"
-Requires-Dist: PyMySQL==1.*; extra == "db"
+Requires-Dist: pg8000<2,>=1; extra == "db"
+Requires-Dist: PyMySQL<2,>=1; extra == "db"
 Requires-Dist: pyodbc<6,>=4; extra == "db"
 Requires-Dist: rdflib<8,>=5; extra == "db"
 Provides-Extra: efm
-Requires-Dist: aniso8601==10.*; extra == "efm"
-Requires-Dist: holidays==0.*; extra == "efm"
-Requires-Dist: Matplotlib==3.*; extra == "efm"
+Requires-Dist: aniso8601<11,>=10; extra == "efm"
+Requires-Dist: holidays<1,>=0; extra == "efm"
+Requires-Dist: matplotlib<4,>=3; extra == "efm"
 Requires-Dist: pytz; extra == "efm"
 Provides-Extra: esef
-Requires-Dist: tinycss2==1.*; extra == "esef"
+Requires-Dist: tinycss2<2,>=1; extra == "esef"
 Provides-Extra: objectmaker
-Requires-Dist: graphviz==0.*; extra == "objectmaker"
+Requires-Dist: graphviz<1,>=0; extra == "objectmaker"
 Provides-Extra: webserver
 Requires-Dist: cheroot<11,>=8; extra == "webserver"
-Requires-Dist: tornado==6.*; extra == "webserver"
+Requires-Dist: tornado<7,>=6; extra == "webserver"
+Provides-Extra: xule
+Requires-Dist: aniso8601<11,>=10; extra == "xule"
 Dynamic: license-file
 
 # Arelle

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/ModelDocument.py

@@ -1579,12 +1579,6 @@ def inlineIxdsDiscover(modelXbrl, modelIxdsDocument, setTargetModelXbrl=False, *
     modelXbrl.targetRoleRefs = {} # roleRefs used by selected target
     modelXbrl.targetArcroleRefs = {}  # arcroleRefs used by selected target
     modelXbrl.targetRelationships = set() # relationship elements used by selected target
-    targetModelXbrl = modelXbrl if setTargetModelXbrl else None # modelXbrl of target for contexts/units in multi-target/multi-instance situation
-    assignUnusedContextsUnits = (not setTargetModelXbrl and not ixdsTarget and
-                                 not getattr(modelXbrl, "supplementalModelXbrls", ()) and (
-                                    not getattr(modelXbrl, "targetIXDSesToLoad", ()) or
-                                    set(e.modelDocument for e in modelXbrl.ixdsHtmlElements) ==
-                                    set(x.modelDocument for e in getattr(modelXbrl, "targetIXDSesToLoad", ()) for x in e[1])))
     hasResources = hasHeader = False
     for htmlElement in modelXbrl.ixdsHtmlElements:
         mdlDoc = htmlElement.modelDocument
@@ -1611,8 +1605,12 @@ def inlineIxdsDiscover(modelXbrl, modelIxdsDocument, setTargetModelXbrl=False, *
         for modelInlineFact in htmlElement.iterdescendants(ixNStag + "nonNumeric", ixNStag + "nonFraction", ixNStag + "fraction"):
             if isinstance(modelInlineFact,ModelObject):
                 _target = modelInlineFact.get("target")
-                factTargetContextRefs[_target].add(modelInlineFact.get("contextRef"))
-                factTargetUnitRefs[_target].add(modelInlineFact.get("unitRef"))
+                contextRef = modelInlineFact.get("contextRef")
+                if contextRef is not None:
+                    factTargetContextRefs[_target].add(contextRef.strip())
+                unitRef = modelInlineFact.get("unitRef")
+                if unitRef is not None:
+                    factTargetUnitRefs[_target].add(unitRef.strip())
                 if modelInlineFact.id:
                     factsByFactID[modelInlineFact.id] = modelInlineFact
         for elt in htmlElement.iterdescendants(tag=ixNStag + "continuation"):
@@ -1734,9 +1732,9 @@ def inlineIxdsDiscover(modelXbrl, modelIxdsDocument, setTargetModelXbrl=False, *
                     targetRoleUris[_target].add(footnoteRole)
 
     contextRefs = factTargetContextRefs[ixdsTarget]
+    contextRefsForAllTargets = {ref for refs in factTargetContextRefs.values() for ref in refs}
     unitRefs = factTargetUnitRefs[ixdsTarget]
-    allContextRefs = set.union(*factTargetContextRefs.values())
-    allUnitRefs = set.union(*factTargetUnitRefs.values())
+    unitRefsForAllTargets = {ref for refs in factTargetUnitRefs.values() for ref in refs}
 
     # discovery of contexts, units and roles which are used by target document
     for htmlElement in modelXbrl.ixdsHtmlElements:
@@ -1745,13 +1743,17 @@ def inlineIxdsDiscover(modelXbrl, modelIxdsDocument, setTargetModelXbrl=False, *
 
         for inlineElement in htmlElement.iterdescendants(tag=ixNStag + "resources"):
             for elt in inlineElement.iterchildren("{http://www.xbrl.org/2003/instance}context"):
-                id = elt.get("id")
-                if id in contextRefs or (assignUnusedContextsUnits and id not in allContextRefs):
+                contextId = elt.get("id")
+                if contextId in contextRefs:
                     modelIxdsDocument.contextDiscover(elt, setTargetModelXbrl)
+                elif contextId not in contextRefsForAllTargets:
+                    modelXbrl.ixdsUnmappedContexts[contextId] = elt
             for elt in inlineElement.iterchildren("{http://www.xbrl.org/2003/instance}unit"):
-                id = elt.get("id")
-                if id in unitRefs or (assignUnusedContextsUnits and id not in allUnitRefs):
+                unitId = elt.get("id")
+                if unitId in unitRefs:
                     modelIxdsDocument.unitDiscover(elt, setTargetModelXbrl)
+                elif unitId not in unitRefsForAllTargets:
+                    modelXbrl.ixdsUnmappedUnits[unitId] = elt
             for refElement in inlineElement.iterchildren("{http://www.xbrl.org/2003/linkbase}roleRef"):
                 r = refElement.get("roleURI")
                 if r in targetRoleUris[ixdsTarget]:

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/ModelInstanceObject.py

@@ -1159,7 +1159,7 @@ class ModelContext(ModelObject):
             self._dimsHash = hash( frozenset(self.qnameDims.values()) )
             return self._dimsHash
 
-    def nonDimValues(self, contextElement):
+    def nonDimValues(self, contextElement: str | int) -> list[ModelObject]:
         """([ModelObject]) -- ContextElement is either string or Aspect code for segment or scenario, returns nonXDT ModelObject children of context element.
 
         :param contextElement: one of 'segment', 'scenario', Aspect.NON_XDT_SEGMENT, Aspect.NON_XDT_SCENARIO, Aspect.COMPLETE_SEGMENT, Aspect.COMPLETE_SCENARIO

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/ModelXbrl.py

@@ -12,7 +12,7 @@ from collections import defaultdict
 from typing import TYPE_CHECKING, Any, TypeVar, Union, cast, Optional
 
 import regex as re
-from collections.abc import Iterable
+from collections.abc import Iterable, Iterator
 
 import arelle
 from arelle import FileSource, ModelRelationshipSet, XmlUtil, ModelValue, XbrlConst, XmlValidate
@@ -335,8 +335,12 @@ class ModelXbrl:
         self.facts: list[ModelFact] = []
         self.factsInInstance: set[ModelFact] = set()
         self.undefinedFacts: list[ModelFact] = []  # elements presumed to be facts but not defined
-        self.contexts: dict[str, ModelDocumentClass.xmlRootElement] = {}
+        self.contexts: dict[str, ModelContext] = {}
+        self.ixdsUnmappedContexts: dict[str, ModelContext] = {}
+        self._contextsInUseMarked = False
         self.units: dict[str, ModelUnit] = {}
+        self.ixdsUnmappedUnits: dict[str, ModelUnit] = {}
+        self._unitsInUseMarked = False
         self.modelObjects: list[ModelObject] = []
         self.qnameParameters: dict[QName, Any] = {}
         self.modelVariableSets: set[ModelVariableSet] = set()
@@ -604,7 +608,7 @@ class ModelXbrl:
                         for cOCCs,mOCCs in ((c.nonDimValues(segAspect),segOCCs),
                                             (c.nonDimValues(scenAspect),scenOCCs)))
                 ):
-                    return cast('ModelContext', c)
+                    return c
         return None
 
     def createContext(
@@ -868,17 +872,24 @@ class ModelXbrl:
             return fbdq[memQname]
 
     @property
-    def contextsInUse(self) -> Any:
-        try:
-            if self._contextsInUseMarked:
-                return (cntx for cntx in self.contexts.values() if getattr(cntx, "_inUse", False))
-        except AttributeError:
+    def contextsInUse(self) -> Iterator[ModelContext]:
+        if not self._contextsInUseMarked:
             for fact in self.factsInInstance:
                 cntx = fact.context
                 if cntx is not None:
                     cntx._inUse = True
-            self._contextsInUseMarked: bool = True
-            return self.contextsInUse
+            self._contextsInUseMarked = True
+        return (cntx for cntx in self.contexts.values() if getattr(cntx, "_inUse", False))
+
+    @property
+    def unitsInUse(self) -> Iterator[ModelUnit]:
+        if not self._unitsInUseMarked:
+            for fact in self.factsInInstance:
+                unit = fact.unit
+                if unit is not None:
+                    unit._inUse = True
+            self._unitsInUseMarked = True
+        return (unit for unit in self.units.values() if getattr(unit, "_inUse", False))
 
     @property
     def dimensionsInUse(self) -> set[Any]:

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/WebCache.py

@@ -6,20 +6,28 @@ 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
+from filelock import FileLock, Timeout
 
 from arelle.PythonUtil import isLegacyAbs
 
@@ -567,14 +575,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_release-2.37.49 → arelle_release-2.37.50}/arelle/_version.py

@@ -17,5 +17,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '2.37.49'
-__version_tuple__ = version_tuple = (2, 37, 49)
+__version__ = version = '2.37.50'
+__version_tuple__ = version_tuple = (2, 37, 50)

{arelle_release-2.37.49 → arelle_release-2.37.50}/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__(

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/DBA/PluginValidationDataExtension.py

@@ -146,7 +146,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:

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/EDINET/Constants.py

@@ -14,12 +14,21 @@ class FormType(Enum):
     FORM_2_4 = '第二号の四様式'
     FORM_2_7 = '第二号の七様式'
     FORM_3 = '第三号様式'
+    FORM_4 = '第四号様式'
+
+    @property
+    def isStockReport(self) -> bool:
+        return self in STOCK_REPORT_FORMS
 
 CORPORATE_FORMS =frozenset([
     FormType.FORM_2_4,
     FormType.FORM_2_7,
     FormType.FORM_3,
 ])
+STOCK_REPORT_FORMS = frozenset([
+    FormType.FORM_3,
+    FormType.FORM_4,
+])
 qnEdinetManifestInsert = qname("{http://disclosure.edinet-fsa.go.jp/2013/manifest}insert")
 qnEdinetManifestInstance = qname("{http://disclosure.edinet-fsa.go.jp/2013/manifest}instance")
 qnEdinetManifestItem = qname("{http://disclosure.edinet-fsa.go.jp/2013/manifest}item")

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/EDINET/PluginValidationDataExtension.py

@@ -24,7 +24,7 @@ from arelle.ValidateDuplicateFacts import getDeduplicatedFacts, DeduplicationTyp
 from arelle.XmlValidate import VALID
 from arelle.typing import TypeGetText
 from arelle.utils.PluginData import PluginData
-from .Constants import CORPORATE_FORMS
+from .Constants import CORPORATE_FORMS, FormType
 from .ControllerPluginData import ControllerPluginData
 from .ManifestInstance import ManifestInstance
 from .Statement import Statement, STATEMENTS, BalanceSheet, StatementInstance, StatementType
@@ -54,7 +54,7 @@ class PluginValidationDataExtension(PluginData):
     def __init__(self, name: str):
         super().__init__(name)
         jpcrpEsrNamespace = "http://disclosure.edinet-fsa.go.jp/taxonomy/jpcrp-esr/2024-11-01/jpcrp-esr_cor"
-        jpcrpNamespace = 'http://disclosure.edinet-fsa.go.jp/taxonomy/jpcrp/2024-11-01/jpcrp_cor'
+        self.jpcrpNamespace = 'http://disclosure.edinet-fsa.go.jp/taxonomy/jpcrp/2024-11-01/jpcrp_cor'
         jpdeiNamespace = 'http://disclosure.edinet-fsa.go.jp/taxonomy/jpdei/2013-08-31/jpdei_cor'
         jpigpNamespace = "http://disclosure.edinet-fsa.go.jp/taxonomy/jpigp/2024-11-01/jpigp_cor"
         jppfsNamespace = "http://disclosure.edinet-fsa.go.jp/taxonomy/jppfs/2024-11-01/jppfs_cor"
@@ -63,11 +63,12 @@ class PluginValidationDataExtension(PluginData):
         self.assetsIfrsQn = qname(jpigpNamespace, 'AssetsIFRS')
         self.consolidatedOrNonConsolidatedAxisQn = qname(jppfsNamespace, 'ConsolidatedOrNonConsolidatedAxis')
         self.documentTypeDeiQn = qname(jpdeiNamespace, 'DocumentTypeDEI')
+        self.issuedSharesTotalNumberOfSharesEtcQn = qname(self.jpcrpNamespace, 'IssuedSharesTotalNumberOfSharesEtcTextBlock')
         self.jpcrpEsrFilingDateCoverPageQn = qname(jpcrpEsrNamespace, 'FilingDateCoverPage')
-        self.jpcrpFilingDateCoverPageQn = qname(jpcrpNamespace, 'FilingDateCoverPage')
+        self.jpcrpFilingDateCoverPageQn = qname(self.jpcrpNamespace, 'FilingDateCoverPage')
         self.jpspsFilingDateCoverPageQn = qname(jpspsNamespace, 'FilingDateCoverPage')
         self.nonConsolidatedMemberQn = qname(jppfsNamespace, "NonConsolidatedMember")
-        self.ratioOfFemaleDirectorsAndOtherOfficersQn = qname(jpcrpNamespace, "RatioOfFemaleDirectorsAndOtherOfficers")
+        self.ratioOfFemaleDirectorsAndOtherOfficersQn = qname(self.jpcrpNamespace, "RatioOfFemaleDirectorsAndOtherOfficers")
 
         self.contextIdPattern = regex.compile(r'(Prior[1-9]Year|CurrentYear|Prior[1-9]Interim|Interim)(Duration|Instant)')
 
@@ -103,6 +104,12 @@ class PluginValidationDataExtension(PluginData):
             return True
         return False
 
+    def isCorporateReport(self, modelXbrl: ModelXbrl) -> bool:
+        return self.jpcrpNamespace in modelXbrl.namespaceDocs
+
+    def isStockForm(self, modelXbrl: ModelXbrl) -> bool:
+        documentTypes = self.getDocumentTypes(modelXbrl)
+        return any(documentType == form.value for form in FormType if form.isStockReport for documentType in documentTypes)
 
     def getBalanceSheets(self, modelXbrl: ModelXbrl, statement: Statement) -> list[BalanceSheet]:
         """

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/EDINET/rules/contexts.py

@@ -4,6 +4,7 @@ See COPYRIGHT.md for copyright information.
 from __future__ import annotations
 
 from collections import defaultdict
+from itertools import chain
 from typing import Any, Iterable
 
 from arelle import XbrlConst
@@ -161,8 +162,9 @@ def rule_EC8054W(
     EDINET.EC8054W: For any context with ID containing "NonConsolidatedMember",
     the scenario element within must be set to "NonConsolidatedMember".
     """
-    for context in val.modelXbrl.contexts.values():
-        if pluginData.nonConsolidatedMemberQn.localName not in context.id:
+    allContexts = chain(val.modelXbrl.contexts.values(), val.modelXbrl.ixdsUnmappedContexts.values())
+    for context in allContexts:
+        if context.id is None or pluginData.nonConsolidatedMemberQn.localName not in context.id:
             continue
         member = context.dimMemberQname(
             pluginData.consolidatedOrNonConsolidatedAxisQn,

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/EDINET/rules/edinet.py

@@ -281,3 +281,25 @@ def rule_EC8075W(
                 codes='EDINET.EC8075W',
                 msg=_("The percentage of female executives has not been tagged in detail."),
                 )
+
+
+@validation(
+    hook=ValidationHook.XBRL_FINALLY,
+    disclosureSystems=[DISCLOSURE_SYSTEM_EDINET],
+)
+def rule_EC8076W(
+        pluginData: PluginValidationDataExtension,
+        val: ValidateXbrl,
+        *args: Any,
+        **kwargs: Any,
+) -> Iterable[Validation]:
+    """
+    EDINET.EC8076W: "Issued Shares, Total Number of Shares, etc. [Text Block]" (IssuedSharesTotalNumberOfSharesEtcTextBlock) is not tagged.
+    Applies to forms 3 and 4.
+    """
+    if pluginData.isStockForm(val.modelXbrl) and pluginData.isCorporateReport(val.modelXbrl):
+        if not pluginData.hasValidNonNilFact(val.modelXbrl, pluginData.issuedSharesTotalNumberOfSharesEtcQn):
+            yield Validation.warning(
+                codes='EDINET.EC8076W',
+                msg=_('"Issued Shares, Total Number of Shares, etc. [Text Block]" (IssuedSharesTotalNumberOfSharesEtcTextBlock) is not tagged.'),
+            )

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/EDINET/rules/gfm.py

@@ -257,13 +257,14 @@ def rule_gfm_1_2_8(
     EDINET.EC5700W: [GFM 1.2.8] Every xbrli:context element must appear in at least one
     contextRef attribute in the same instance.
     """
-    unused_contexts = list(set(val.modelXbrl.contexts.values()) - set(val.modelXbrl.contextsInUse))
-    unused_contexts.sort(key=lambda x: x.id)
-    for context in unused_contexts:
+    unusedContexts = list(set(val.modelXbrl.contexts.values()) - set(val.modelXbrl.contextsInUse))
+    unusedContexts.extend(val.modelXbrl.ixdsUnmappedContexts.values())
+    unusedContexts.sort(key=lambda x: x.id if x.id is not None else "")
+    for context in unusedContexts:
         yield Validation.warning(
             codes='EDINET.EC5700W.GFM.1.2.8',
             msg=_('If you are not using a context, delete it if it is not needed.'),
-            modelObject = context
+            modelObject=context
         )
 
 
@@ -484,6 +485,7 @@ def rule_gfm_1_2_25(
             XbrlConst.qnXbrliEndDate.clarkNotation,
             XbrlConst.qnXbrliInstant.clarkNotation
         ):
+            elt = cast(ModelObject, elt)
             dateText = XmlUtil.text(elt)
             if not GFM_CONTEXT_DATE_PATTERN.match(dateText):
                 errors.append(elt)
@@ -558,12 +560,14 @@ def rule_gfm_1_2_27(
     EDINET.EC5700W: [GFM 1.2.27] An instance must not contain unused units.
     """
     # TODO: Consolidate validations involving unused units
-    unusedUnits = set(val.modelXbrl.units.values()) - {fact.unit for fact in val.modelXbrl.facts if fact.unit is not None}
+    unusedUnits = list(set(val.modelXbrl.units.values()) - set(val.modelXbrl.unitsInUse))
+    unusedUnits.extend(val.modelXbrl.ixdsUnmappedUnits.values())
+    unusedUnits.sort(key=lambda x: x.hash)
     if len(unusedUnits) > 0:
         yield Validation.warning(
             codes='EDINET.EC5700W.GFM.1.2.27',
             msg=_("Delete unused units from the instance."),
-            modelObject=list(unusedUnits)
+            modelObject=unusedUnits
         )
 
 

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/ESEF/ESEF_2021/ValidateXbrlFinally.py

@@ -529,11 +529,11 @@ def validateXbrlFinally(val: ValidateXbrl, *args: Any, **kwargs: Any) -> None:
         if contextsWithDisallowedOCEs:
             modelXbrl.error("ESEF.2.1.3.segmentUsed",
                 _("xbrli:segment container MUST NOT be used in contexts: %(contextIds)s"),
-                modelObject=contextsWithDisallowedOCEs, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEs))
+                modelObject=contextsWithDisallowedOCEs, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEs if c.id is not None))
         if contextsWithDisallowedOCEcontent:
             modelXbrl.error("ESEF.2.1.3.scenarioContainsNonDimensionalContent",
                 _("xbrli:scenario in contexts MUST NOT contain any other content than defined in XBRL Dimensions specification: %(contextIds)s"),
-                modelObject=contextsWithDisallowedOCEcontent, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEcontent))
+                modelObject=contextsWithDisallowedOCEcontent, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEcontent if c.id is not None))
         if len(contextIdentifiers) > 1:
             modelXbrl.error("ESEF.2.1.4.multipleIdentifiers",
                 _("All entity identifiers in contexts MUST have identical content: %(contextIds)s"),

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/ESEF/ESEF_Current/ValidateXbrlFinally.py

@@ -589,11 +589,11 @@ def validateXbrlFinally(val: ValidateXbrl, *args: Any, **kwargs: Any) -> None:
         if contextsWithDisallowedOCEs:
             modelXbrl.error("ESEF.2.1.3.segmentUsed",
                 _("xbrli:segment container MUST NOT be used in contexts: %(contextIds)s"),
-                modelObject=contextsWithDisallowedOCEs, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEs))
+                modelObject=contextsWithDisallowedOCEs, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEs if c.id is not None))
         if contextsWithDisallowedOCEcontent:
             modelXbrl.error("ESEF.2.1.3.scenarioContainsNonDimensionalContent",
                 _("xbrli:scenario in contexts MUST NOT contain any other content than defined in XBRL Dimensions specification: %(contextIds)s"),
-                modelObject=contextsWithDisallowedOCEcontent, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEcontent))
+                modelObject=contextsWithDisallowedOCEcontent, contextIds=", ".join(c.id for c in contextsWithDisallowedOCEcontent if c.id is not None))
         if len(contextIdentifiers) > 1:
             modelXbrl.error("ESEF.2.1.4.multipleIdentifiers",
                 _("All entity identifiers in contexts MUST have identical content: %(contextIds)s"),

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/NL/PluginValidationDataExtension.py

@@ -265,7 +265,7 @@ class PluginValidationDataExtension(PluginData):
                             contextsWithPeriodTimeZone.append(context)
                 if context.hasSegment:
                     contextsWithSegments.append(context)
-                if context.nonDimValues("scenario"):  # type: ignore[no-untyped-call]
+                if context.nonDimValues("scenario"):
                     contextsWithImproperContent.append(context)
         return ContextData(
             contextsWithImproperContent=contextsWithImproperContent,

{arelle_release-2.37.49 → arelle_release-2.37.50}/arelle/plugin/validate/NL/rules/fr_nl.py

@@ -552,9 +552,9 @@ def rule_fr_nl_3_03(
     """
     FR-NL-3.03: An XBRL instance document MUST NOT contain unused contexts
     """
-    unused_contexts = list(set(val.modelXbrl.contexts.values()) - set(val.modelXbrl.contextsInUse))
-    unused_contexts.sort(key=lambda x: x.id)
-    for context in unused_contexts:
+    unusedContexts = list(set(val.modelXbrl.contexts.values()) - set(val.modelXbrl.contextsInUse))
+    unusedContexts.sort(key=lambda x: x.id if x.id is not None else "")
+    for context in unusedContexts:
         yield Validation.error(
             codes='NL.FR-NL-3.03',
             msg=_('Unused context must not exist in XBRL instance document'),
@@ -637,10 +637,9 @@ def rule_fr_nl_4_02(
     """
     FR-NL-4.02: An XBRL instance document MUST NOT contain unused 'xbrli:unit' elements
     """
-    unused_units_set = set(val.modelXbrl.units.values()) - {fact.unit for fact in val.modelXbrl.facts if fact.unit is not None}
-    unused_units = list(unused_units_set)
-    unused_units.sort(key=lambda x: x.hash)
-    for unit in unused_units:
+    unusedUnits = list(set(val.modelXbrl.units.values()) - set(val.modelXbrl.unitsInUse))
+    unusedUnits.sort(key=lambda x: x.hash)
+    for unit in unusedUnits:
         yield Validation.error(
             codes='NL.FR-NL-4.02',
             msg=_('Unused unit must not exist in the XBRL instance document'),