ezdxf 1.3.4rc0__zip → 1.3.5b0__zip

{ezdxf-1.3.4rc0/src/ezdxf.egg-info → ezdxf-1.3.5b0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: ezdxf
-Version: 1.3.4rc0
+Version: 1.3.5b0
 Summary: A Python package to create/manipulate DXF drawings.
 Author-email: Manfred Moitzi <me@mozman.at>
 Project-URL: Repository, https://github.com/mozman/ezdxf

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/addons/odafc.py

@@ -23,8 +23,6 @@ from ezdxf.lldxf.validator import (
 )
 
 logger = logging.getLogger("ezdxf")
-win_exec_path = ezdxf.options.get("odafc-addon", "win_exec_path").strip('"')
-unix_exec_path = ezdxf.options.get("odafc-addon", "unix_exec_path").strip('"')
 
 
 class ODAFCError(IOError):
@@ -93,14 +91,23 @@ LINUX = "Linux"
 DARWIN = "Darwin"
 
 
+def get_win_exec_path() -> str:
+    return ezdxf.options.get("odafc-addon", "win_exec_path").strip('"')
+
+
+def get_unix_exec_path() -> str:
+    return ezdxf.options.get("odafc-addon", "unix_exec_path").strip('"')
+
+
 def is_installed() -> bool:
     """Returns ``True`` if the ODAFileConverter is installed."""
     if platform.system() in (LINUX, DARWIN):
+        unix_exec_path = get_unix_exec_path()
         if unix_exec_path and Path(unix_exec_path).is_file():
             return True
         return shutil.which("ODAFileConverter") is not None
     # Windows:
-    return os.path.exists(win_exec_path)
+    return os.path.exists(get_win_exec_path())
 
 
 def map_version(version: str) -> str:
@@ -371,6 +378,8 @@ def _get_odafc_path(system: str) -> str:
     # return this path as exec path.
     # This may help if the "which" command can not find the "ODAFileConverter"
     # command and also adds support for AppImages provided by ODA.
+    unix_exec_path = get_unix_exec_path()
+
     if system != WINDOWS and unix_exec_path:
         if Path(unix_exec_path).is_file():
             return unix_exec_path
@@ -381,7 +390,7 @@ def _get_odafc_path(system: str) -> str:
 
     path = shutil.which("ODAFileConverter")
     if not path and system == WINDOWS:
-        path = win_exec_path
+        path = get_win_exec_path()
         if not Path(path).is_file():
             path = None
 
@@ -434,7 +443,9 @@ def _run_with_no_gui(
         with _linux_dummy_display() as display:
             env = os.environ.copy()
             env["DISPLAY"] = display
-            proc = subprocess.run([command] + arguments, text=True, capture_output=True, env=env)
+            proc = subprocess.run(
+                [command] + arguments, text=True, capture_output=True, env=env
+            )
 
     elif system == DARWIN:
         # TODO: unknown how to prevent the GUI from appearing on macOS

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/addons/openscad.py

@@ -16,18 +16,13 @@ from ezdxf.render import MeshBuilder, MeshTransformer
 from ezdxf.addons import meshex
 
 
-DEFAULT_WIN_OPENSCAD_PATH = ezdxf.options.get(
-    "openscad-addon", "win_exec_path"
-).strip('"')
-CMD = "openscad"
-
-
 class Operation(enum.Enum):
     union = 0
     difference = 1
     intersection = 2
 
 
+CMD = "openscad"
 UNION = Operation.union
 DIFFERENCE = Operation.difference
 INTERSECTION = Operation.intersection
@@ -37,16 +32,21 @@ def get_openscad_path() -> str:
     if platform.system() in ("Linux", "Darwin"):
         return CMD
     else:
-        return DEFAULT_WIN_OPENSCAD_PATH
+        return ezdxf.options.get("openscad-addon", "win_exec_path").strip('"')
 
 
 def is_installed() -> bool:
-    r"""Returns ``True`` if OpenSCAD is installed. On Windows only the
-    default install path 'C:\\Program Files\\OpenSCAD\\openscad.exe' is checked.
+    """Returns ``True`` if OpenSCAD is installed.
+
+    Searches on Windows the path stored in the options as "win_exec_path" in section
+    "[openscad-addon]" which is "C:\\Program Files\\OpenSCAD\\openscad.exe" by default.
+
+    Searches the "openscad" command on Linux and macOS.
+
     """
     if platform.system() in ("Linux", "Darwin"):
         return shutil.which(CMD) is not None
-    return Path(DEFAULT_WIN_OPENSCAD_PATH).exists()
+    return Path(get_openscad_path()).exists()
 
 
 def run(script: str, exec_path: Optional[str] = None) -> MeshTransformer:
@@ -94,7 +94,7 @@ def run(script: str, exec_path: Optional[str] = None) -> MeshTransformer:
 def str_matrix44(m: Matrix44) -> str:
     # OpenSCAD uses column major order!
     import numpy as np
-    
+
     def cleanup(values: Iterable) -> Iterable:
         for value in values:
             if isinstance(value, np.float64):
@@ -295,9 +295,7 @@ class Script:
         return "\n".join(self.data)
 
 
-def boolean_operation(
-    op: Operation, mesh1: MeshBuilder, mesh2: MeshBuilder
-) -> str:
+def boolean_operation(op: Operation, mesh1: MeshBuilder, mesh2: MeshBuilder) -> str:
     """Returns an `OpenSCAD`_ script to apply the given boolean operation to the
     given meshes.
 

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/audit.py

@@ -42,6 +42,8 @@ class AuditError(IntEnum):
     MISPLACED_ROOT_DICT = 14
     ROOT_DICT_NOT_FOUND = 15
     REMOVED_ENTITY_WITH_INVALID_OWNER_HANDLE = 16
+    MODELSPACE_NOT_FOUND = 17
+    ACTIVE_PAPERSPACE_LAYOUT_NOT_FOUND = 18
 
     UNDEFINED_LINETYPE = 100
     UNDEFINED_DIMENSION_STYLE = 101
@@ -91,6 +93,7 @@ class AuditError(IntEnum):
     INVALID_CREASE_VALUE_COUNT = 224
     INVALID_ELLIPSE_RATIO = 225
     INVALID_HATCH_BOUNDARY_PATH = 226
+    TAG_ATTRIBUTE_MISSING = 227
 
 
 REQUIRED_ROOT_DICT_ENTRIES = ("ACAD_GROUP", "ACAD_PLOTSTYLENAME")
@@ -109,7 +112,7 @@ class ErrorEntry:
         self.message: str = message  # error message
         self.data: Any = data  # additional data as an arbitrary object
 
-# pylint: disable=too-many-public-methods
+
 class Auditor:
     def __init__(self, doc: Drawing) -> None:
         assert doc is not None and doc.rootdict is not None and doc.entitydb is not None
@@ -237,6 +240,8 @@ class Auditor:
             return self.errors
         self.doc.entitydb.audit(self)
         self.check_root_dict_entries()
+        self.check_modelspace_exist()
+        self.check_active_layout_exist()
         self.check_tables()
         self.doc.objects.audit(self)
         self.doc.blocks.audit(self)
@@ -296,6 +301,26 @@ class Auditor:
                     dxf_entity=rootdict,
                 )
 
+    def check_modelspace_exist(self) -> None:
+        msp = self.doc.modelspace()
+        if not msp.is_alive:
+            self.add_error(
+                code=AuditError.MODELSPACE_NOT_FOUND,
+                message=f"Required modelspace layout not found.",
+            )
+
+    def check_active_layout_exist(self) -> None:
+        try:
+            layout = self.doc.active_layout()
+        except const.DXFStructureError:
+            layout = None  # type: ignore
+
+        if layout is None or layout.is_alive is False:
+            self.add_error(
+                code=AuditError.ACTIVE_PAPERSPACE_LAYOUT_NOT_FOUND,
+                message=f"Required active paperspace layout not found.",
+            )
+
     def check_tables(self) -> None:
         table_section = self.doc.tables
         table_section.viewports.audit(self)

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/blkrefs.py

@@ -1,4 +1,4 @@
-#  Copyright (c) 2021-2022, Manfred Moitzi
+#  Copyright (c) 2021-2024, Manfred Moitzi
 #  License: MIT License
 from __future__ import annotations
 from typing import TYPE_CHECKING, Iterable, Optional, Iterator
@@ -12,7 +12,7 @@ if TYPE_CHECKING:
     from ezdxf.lldxf.tags import Tags
     from ezdxf.entities import DXFEntity, BlockRecord
 
-__all__ = ["BlockDefinitionIndex", "BlockReferenceCounter"]
+__all__ = ["BlockDefinitionIndex", "BlockReferenceCounter", "find_unreferenced_blocks"]
 
 """ 
 Where are block references located:
@@ -207,3 +207,34 @@ def header_section_handles(doc: "Drawing") -> Iterable[str]:
             block = doc.blocks.get(blk_name, None)
             if block is not None:
                 yield block.block_record.dxf.handle
+
+
+def find_unreferenced_blocks(doc: Drawing) -> set[str]:
+    """Returns the names of all block definitions without references.
+
+    .. warning::
+
+        The DXF reference does not document all uses of blocks. The INSERT entity is
+        just one explicit use case, but there are also many indirect block references
+        and the customizability of DXF allows you to store block names and handles in
+        many places.
+
+        There are some rules for storing names and handles and this module checks all of
+        these known rules, but there is no guarantee that everyone follows these rules.
+
+        Therefore, it is still possible to destroy a DXF document by deleting an
+        absolutely necessary block definition.
+
+    .. versionadded:: 1.3.5
+
+    """
+    ref_counter = BlockReferenceCounter(doc)
+    unreferenced_blocks: set[str] = set()
+
+    for block in doc.blocks:
+        if not block.is_alive or block.is_any_layout:
+            continue
+        count = ref_counter.by_name(block.name)
+        if count == 0:
+            unreferenced_blocks.add(block.name)
+    return unreferenced_blocks

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/entities/__init__.py

@@ -100,7 +100,11 @@ from .mleader import MLeader, MLeaderStyle, MLeaderStyleCollection, MultiLeader
 # register graphical entities R2007
 
 from .light import Light
-from .acad_table import AcadTableBlockContent, acad_table_to_block
+from .acad_table import (
+    AcadTableBlockContent,
+    acad_table_to_block,
+    read_acad_table_content,
+)
 
 # register graphical entities R2010
 

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/entities/attrib.py

@@ -4,6 +4,8 @@ from __future__ import annotations
 from typing import TYPE_CHECKING, Optional
 from typing_extensions import Self
 import copy
+
+from ezdxf.audit import Auditor, AuditError
 from ezdxf.lldxf import validator
 from ezdxf.math import NULLVEC, Vec3, Z_AXIS, OCS, Matrix44
 from ezdxf.lldxf.attributes import (
@@ -40,7 +42,6 @@ from .copy import default_copy
 if TYPE_CHECKING:
     from ezdxf.lldxf.tagwriter import AbstractTagWriter
     from ezdxf.lldxf.tags import Tags
-    from ezdxf.entities import DXFEntity
     from ezdxf import xref
 
 
@@ -81,6 +82,7 @@ attrib_fields = {
     # ezdxf stores the last group code 280 as "lock_position" attribute and does
     # not export a version tag for any DXF version.
     # Tag string (cannot contain spaces):
+    # Mandatory by AutoCAD!
     "tag": DXFAttr(
         2,
         default="",
@@ -118,7 +120,7 @@ attrib_fields = {
     # 4 = multiline ATTDEF
     "attribute_type": DXFAttr(
         71,
-        default=1,
+        default=const.ATTRIB_TYPE_SINGLE_LINE,
         dxfversion=const.DXF2018,
         optional=True,
         validator=validator.is_one_of({1, 2, 4}),
@@ -345,6 +347,14 @@ class BaseAttrib(Text):
         self._embedded_mtext.set_mtext(mtext)
         _update_content_from_mtext(self, mtext)
         _update_location_from_mtext(self, mtext)
+
+        # set attribute type:
+        if isinstance(self, Attrib):
+            attribute_type = const.ATTRIB_TYPE_MULTI_LINE
+        else:
+            attribute_type = const.ATTDEF_TYPE_MULTI_LINE
+        self.dxf.attribute_type = attribute_type
+
         # misc properties
         self.dxf.style = mtext.dxf.style
         self.dxf.height = mtext.dxf.char_height
@@ -356,7 +366,7 @@ class BaseAttrib(Text):
 
     def embed_mtext(self, mtext: MText, graphic_properties=True) -> None:
         """Set multi-line properties from a :class:`MText` entity and destroy the
-        source entity afterwards.
+        source entity afterward.
 
         The multi-line ATTRIB/ATTDEF entity requires DXF R2018, otherwise an
         ordinary single line ATTRIB/ATTDEF entity will be exported.
@@ -370,6 +380,15 @@ class BaseAttrib(Text):
         self.set_mtext(mtext, graphic_properties)
         mtext.destroy()
 
+    def discard_mtext(self) -> None:
+        """Discard multi-line feature.
+
+        The embedded MTEXT will be removed and the ATTRIB/ATTDEF will be converted to a
+        single-line attribute.
+        """
+        self._embedded_mtext = None
+        self.dxf.attribute_type = const.ATTRIB_TYPE_SINGLE_LINE
+
     def register_resources(self, registry: xref.Registry) -> None:
         """Register required resources to the resource registry."""
         super().register_resources(registry)
@@ -394,6 +413,16 @@ class BaseAttrib(Text):
             self.post_transform(m)
         return self
 
+    def audit(self, auditor: Auditor) -> None:
+        """Validity check."""
+        super().audit(auditor)
+        if not self.dxf.hasattr("tag"):
+            auditor.fixed_error(
+                code=AuditError.TAG_ATTRIBUTE_MISSING,
+                message=f'Missing mandatory "tag" attribute, entity {str(self)} deleted.',
+            )
+            auditor.trash(self)
+
 
 def _update_content_from_mtext(text: Text, mtext: MText) -> None:
     content = mtext.plain_text(split=True, fast=True)
@@ -457,7 +486,6 @@ class AttDef(BaseAttrib):
         self.export_acdb_text(tagwriter)
         self.export_acdb_attdef(tagwriter)
         if tagwriter.dxfversion >= const.DXF2018:
-            self.dxf.attribute_type = 4 if self.has_embedded_mtext_entity else 1
             self.export_dxf_r2018_features(tagwriter)
 
     def export_acdb_attdef(self, tagwriter: AbstractTagWriter) -> None:
@@ -508,7 +536,6 @@ class Attrib(BaseAttrib):
         self.export_acdb_attrib_text(tagwriter)
         self.export_acdb_attrib(tagwriter)
         if tagwriter.dxfversion >= const.DXF2018:
-            self.dxf.attribute_type = 2 if self.has_embedded_mtext_entity else 1
             self.export_dxf_r2018_features(tagwriter)
 
     def export_acdb_attrib_text(self, tagwriter: AbstractTagWriter) -> None:

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/entities/dictionary.py

@@ -48,7 +48,7 @@ acdb_dictionary = DefSubclass(
         "hard_owned": DXFAttr(
             280,
             default=1,
-            optional=True,
+            optional=False,  # 2024-11-18: changed to False because of issue #1203
             validator=validator.is_integer_bool,
             fixer=RETURN_DEFAULT,
         ),

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/entities/dimension.py

@@ -414,7 +414,7 @@ class OverrideMixin:
 
         # transform block record handles into block names
         for attrib_name in ("dimblk", "dimblk1", "dimblk2", "dimldrblk"):
-            blkrec_handle = data.pop(attrib_name + "_handle", None)
+            blkrec_handle = data.pop(attrib_name + "_handle", "")
             if blkrec_handle:
                 set_arrow_name(attrib_name, blkrec_handle)
 
@@ -433,7 +433,7 @@ class OverrideMixin:
 
         # transform linetype handles into LTYPE entry names
         for attrib_name in ("dimltype", "dimltex1", "dimltex2"):
-            handle = data.pop(attrib_name + "_handle", None)
+            handle = data.pop(attrib_name + "_handle", "")
             if handle:
                 set_ltype_name(attrib_name, handle)
         return data
@@ -487,7 +487,7 @@ class Dimension(DXFGraphic, OverrideMixin):
         self.virtual_block_content: Optional[EntitySpace] = None
 
     def copy(self, copy_strategy=default_copy) -> Dimension:
-        virtual_copy = super().copy(copy_strategy=copy_strategy)
+        virtual_copy: Dimension = super().copy(copy_strategy=copy_strategy)  # type: ignore
         # The new virtual copy can not reference the same geometry block as the
         # original dimension entity:
         virtual_copy.dxf.discard("geometry")
@@ -804,10 +804,9 @@ class Dimension(DXFGraphic, OverrideMixin):
 
         ocs = self.ocs()
         dim_elevation = self.dxf.text_midpoint.z
-        transform = False
+        m: Matrix44 | None = None
         insert = self.dxf.get("insert", None)
         if insert:
-            transform = True
             insert = Vec3(ocs.to_wcs(insert))
             m = Matrix44.translate(insert.x, insert.y, insert.z)
 
@@ -823,7 +822,7 @@ class Dimension(DXFGraphic, OverrideMixin):
                 # POINT:
                 ocs_to_wcs(copy, dim_elevation)
 
-            if transform:
+            if m is not None:
                 copy.transform(m)
             yield copy
 

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/entities/insert.py

@@ -12,6 +12,8 @@ from typing import (
 )
 from typing_extensions import Self
 import math
+import logging
+
 from ezdxf.lldxf import validator
 from ezdxf.lldxf.attributes import (
     DXFAttr,
@@ -69,6 +71,7 @@ if TYPE_CHECKING:
 
 __all__ = ["Insert"]
 
+logger = logging.getLogger("ezdxf")
 ABS_TOL = 1e-9
 
 # DXF files as XREF:
@@ -628,8 +631,8 @@ class Insert(LinkedEntities):
         if the property :attr:`Insert.mcount` > 1, use the :meth:`Insert.multi_insert`
         method to resolve the MINSERT entity into multiple INSERT entities.
 
-        This method does not apply the clipping path created by the XCLIP command. 
-        The method returns all entities and ignores the clipping path polygon and no 
+        This method does not apply the clipping path created by the XCLIP command.
+        The method returns all entities and ignores the clipping path polygon and no
         entity is clipped.
 
         The `skipped_entity_callback()` will be called for all entities which are not
@@ -718,18 +721,39 @@ class Insert(LinkedEntities):
                 pairs
 
         """
-
         def unpack(dxfattribs) -> tuple[str, str, UVec]:
             tag = dxfattribs.pop("tag")
-            text = values.get(tag, "")
+            text = values.get(tag, None)
+            if text is None:  # get default value from ATTDEF
+                text = dxfattribs.get("text", "")
             location = dxfattribs.pop("insert")
             return tag, text, location
 
         def autofill() -> None:
             for attdef in block_layout.attdefs():  # type: ignore
                 dxfattribs = attdef.dxfattribs(drop={"prompt", "handle"})
+
+                # Caution! Some mandatory values may not exist!
+                # These are DXF structure errors, but loaded DXF files may have errors!
+                if "tag" not in dxfattribs:
+                    # ATTRIB without "tag" makes no sense!
+                    logger.warning(
+                        f"Skipping {str(attdef)}: missing mandatory 'tag' attribute"
+                    )
+                    continue
+                if "insert" not in dxfattribs:
+                    # Don't know where to place the ATTRIB entity.
+                    logger.warning(
+                        f"Skipping {str(attdef)}: missing mandatory 'insert' attribute"
+                    )
+                    continue
+
                 tag, text, location = unpack(dxfattribs)
                 attrib = self.add_attrib(tag, text, location, dxfattribs)
+                if attdef.has_embedded_mtext_entity:
+                    mtext = attdef.virtual_mtext_entity()
+                    mtext.text = text
+                    attrib.embed_mtext(mtext)
                 attrib.transform(m)
 
         block_layout = self.block()

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/layouts/base.py

@@ -51,9 +51,6 @@ class _AbstractLayout(CreatorInterface):
         """Returns drawing entity database. (internal API)"""
         return self.doc.entitydb
 
-    def rename(self, name) -> None:
-        pass
-
     def __len__(self) -> int:
         """Returns count of entities owned by the layout."""
         return len(self.entity_space)
@@ -134,7 +131,9 @@ class BaseLayout(_AbstractLayout):
     @property
     def is_active_paperspace(self) -> bool:
         """``True`` if is active layout."""
-        return self.block_record.is_active_paperspace
+        if self.block_record.is_alive:
+            return self.block_record.is_active_paperspace
+        return False
 
     @property
     def is_any_paperspace(self) -> bool:

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/layouts/blocklayout.py

@@ -93,7 +93,7 @@ class BlockLayout(BaseLayout):
         """
         return self.get_attdef(tag) is not None
 
-    def get_attdef(self, tag: str) -> Optional[DXFGraphic]:
+    def get_attdef(self, tag: str) -> Optional[AttDef]:
         """Returns attached :class:`~ezdxf.entities.attrib.Attdef` entity by
         `tag` name.
         """

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/layouts/layouts.py

@@ -1,9 +1,9 @@
-# Copyright (c) 2011-2022, Manfred Moitzi
+# Copyright (c) 2011-2024, Manfred Moitzi
 # License: MIT License
 from __future__ import annotations
 from typing import TYPE_CHECKING, Iterator, cast, Optional
 import logging
-from ezdxf.lldxf.const import DXFKeyError, DXFValueError, DXFInternalEzdxfError
+from ezdxf.lldxf.const import DXFKeyError, DXFValueError, DXFStructureError
 from ezdxf.lldxf.const import (
     MODEL_SPACE_R2000,
     PAPER_SPACE_R2000,
@@ -22,8 +22,7 @@ logger = logging.getLogger("ezdxf")
 
 
 def key(name: str) -> str:
-    """AutoCAD uses case insensitive layout names, but stores the name case
-    sensitive."""
+    """AutoCAD uses case-insensitive layout names, but stores the name case-sensitive."""
     return name.upper()
 
 
@@ -205,12 +204,11 @@ class Layouts:
         return cast(Modelspace, self.get("Model"))
 
     def names(self) -> list[str]:
-        """Returns a list of all layout names, all names in original case
-        sensitive form."""
+        """Returns a list of all layout names, all names in original case-sensitive form."""
         return [layout.name for layout in self._layouts.values()]
 
     def get(self, name: Optional[str]) -> Layout:
-        """Returns :class:`~ezdxf.layouts.Layout` by `name`, case insensitive
+        """Returns :class:`~ezdxf.layouts.Layout` by `name`, case-insensitive
         "Model" == "MODEL".
 
         Args:
@@ -226,8 +224,8 @@ class Layouts:
         not exist.
 
         Args:
-            old_name: actual layout name, case insensitive
-            new_name: new layout name, case insensitive
+            old_name: actual layout name, case-insensitive
+            new_name: new layout name, case-insensitive
 
         Raises:
             DXFValueError: try to rename ``'Model'``
@@ -245,7 +243,6 @@ class Layouts:
 
         layout = self.get(old_name)
         self._discard(layout)
-        layout.rename(new_name)
         self._add_layout(new_name, layout)
 
     def names_in_taborder(self) -> list[str]:
@@ -311,7 +308,7 @@ class Layouts:
             name (str): layout name as shown in tabs
 
         Raises:
-            DXFKeyError: if layout `name` do not exists
+            DXFKeyError: if layout `name` do not exist
             DXFValueError: deleting modelspace layout is not possible
             DXFValueError: deleting last paperspace layout is not possible
 
@@ -338,7 +335,7 @@ class Layouts:
         for layout in self:
             if layout.is_active_paperspace:
                 return cast(Paperspace, layout)
-        raise DXFInternalEzdxfError("No active paperspace layout found.")
+        raise DXFStructureError("No active paperspace layout found.")
 
     def audit(self, auditor: Auditor):
         from ezdxf.audit import AuditError
@@ -377,7 +374,7 @@ class Layouts:
                     message=f'Removed orphaned layout {str(br)} "{name}"',
                 )
                 if name in doc.blocks:
-                    doc.blocks.delete_block(name)
+                    doc.blocks.delete_block(name, safe=False)
                 else:
                     doc.block_records.remove(name)
         # Does not check the LAYOUT content this is done in the BlockSection,

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/lldxf/const.py

@@ -570,6 +570,10 @@ ATTRIB_CONST = 2  # This is a constant attribute
 ATTRIB_VERIFY = 4  # Verification is required on input of this attribute
 ATTRIB_IS_PRESET = 8  # no prompt during insertion
 
+ATTRIB_TYPE_SINGLE_LINE = 1
+ATTRIB_TYPE_MULTI_LINE = 2
+ATTDEF_TYPE_MULTI_LINE = 4
+
 # '|' is allowed in layer name, as ltype name ...
 INVALID_NAME_CHARACTERS = '<>/\\":;?*=`'
 INVALID_LAYER_NAME_CHARACTERS = set(INVALID_NAME_CHARACTERS)

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/math/bulge.py

@@ -89,6 +89,12 @@ def bulge_to_arc(
     and 2D :class:`~ezdxf.entities.Polyline` entities start at the vertex which
     includes the bulge value and ends at the following vertex.
 
+    .. important::
+
+        The return values always describe a counter-clockwise oriented arc, so for
+        clockwise arcs (negative bulge values) the start and end angles are swapped and
+        the arc starts at the `end_point` and ends at the `start_point`.
+
     Based on Bulge to Arc by `Lee Mac`_.
 
     Args:

{ezdxf-1.3.4rc0 → ezdxf-1.3.5b0}/src/ezdxf/math/rtree.py

@@ -8,6 +8,7 @@
 # - Research paper of Antonin Guttman:
 #   http://www-db.deis.unibo.it/courses/SI-LS/papers/Gut84.pdf
 from __future__ import annotations
+from operator import itemgetter
 import statistics
 from typing import Iterator, Callable, Sequence, Iterable, TypeVar, Generic
 import abc
@@ -260,7 +261,7 @@ def box_split(points: list[T], max_size: int) -> Sequence[Node[T]]:
     n = len(points)
     size: tuple[float, float, float] = BoundingBox(points).size.xyz
     dim = size.index(max(size))
-    points.sort(key=lambda vec: vec[dim])
+    points.sort(key=itemgetter(dim))
     k = math.ceil(n / max_size)
     return tuple(
         make_node(points[i : i + k], max_size, box_split) for i in range(0, n, k)