BUG: missing error on name without leading /

pypdf/_writer.py

@@ -1898,15 +1898,15 @@ def add_uri(
 
         border_arr: BorderArrayType
         if border is not None:
-            border_arr = [NameObject(n) for n in border[:3]]
+            border_arr = [NumberObject(n) for n in border[:3]]
             if len(border) == 4:
-                dash_pattern = ArrayObject([NameObject(n) for n in border[3]])
+                dash_pattern = ArrayObject([NumberObject(n) for n in border[3]])
                 border_arr.append(dash_pattern)
         else:
             border_arr = [NumberObject(2), NumberObject(2), NumberObject(2)]
 
         if isinstance(rect, str):
-            rect = NameObject(rect)
+            rect = NumberObject(rect)
         elif isinstance(rect, RectangleObject):
             pass
         else:

pypdf/annotations/_markup_annotations.py

@@ -165,8 +165,8 @@ def __init__(
                 ),
                 NameObject("/LE"): ArrayObject(
                     [
-                        NameObject(None),
-                        NameObject(None),
+                        NameObject("/None"),
+                        NameObject("/None"),
                     ]
                 ),
                 NameObject("/IC"): ArrayObject(
@@ -290,7 +290,7 @@ def __init__(
                 NameObject("/Type"): NameObject("/Annot"),
                 NameObject("/Subtype"): NameObject("/Polygon"),
                 NameObject("/Vertices"): ArrayObject(coord_list),
-                NameObject("/IT"): NameObject("PolygonCloud"),
+                NameObject("/IT"): NameObject("/PolygonCloud"),
                 NameObject("/Rect"): RectangleObject(_get_bounding_rectangle(vertices)),
             }
         )

pypdf/generic/_base.py

@@ -615,7 +615,7 @@ def write_to_stream(
     def renumber(self) -> bytes:
         out = self[0].encode("utf-8")
         if out != b"/":
-            logger_warning(f"Incorrect first char in NameObject:({self})", __name__)
+            deprecate_no_replacement(f"Incorrect first char in NameObject, should start with '/':({self})","5.0.0")
         for c in self[1:]:
             if c > "~":
                 for x in c.encode("utf-8"):

tests/test_generic.py

@@ -219,9 +219,8 @@ def test_name_object(caplog):
 
     caplog.clear()
     b = BytesIO()
-    NameObject("hello").write_to_stream(b)
-    assert bytes(b.getbuffer()) == b"hello"
-    assert "Incorrect first char" in caplog.text
+    with pytest.raises(DeprecationWarning):
+        NameObject("hello").write_to_stream(b)
 
     caplog.clear()
     b = BytesIO()