From c6eca2271770308bac2747f0c32030ffaf8d7747 Mon Sep 17 00:00:00 2001 From: Evan Taylor Yates Date: Sat, 3 Oct 2020 17:04:44 -0700 Subject: [PATCH] Add parens around unmarked clause I'm not 100% sure what this sentence is saying, but I'm pretty sure it needs either commas or parentheses to break it up a little. I think parens make the most sense in this case assuming I am reading it correctly. Here's the original sentence and proposed edit in plaintext: ORIG "...persistent objects which were removed from a collection or in some cases a scalar attribute may also be pulled into the Session of a parent object;..." EDIT "...persistent objects which were removed from a collection (or in some cases a scalar attribute) may also be pulled into the Session of a parent object;..." --- doc/build/orm/cascades.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/build/orm/cascades.rst b/doc/build/orm/cascades.rst index 3f06b34c2f..8a2a948760 100644 --- a/doc/build/orm/cascades.rst +++ b/doc/build/orm/cascades.rst @@ -95,7 +95,7 @@ becomes part of the state of that :class:`.Session`:: ``save-update`` has the possibly surprising behavior which is that persistent objects which were *removed* from a collection -or in some cases a scalar attribute +(or in some cases a scalar attribute) may also be pulled into the :class:`.Session` of a parent object; this is so that the flush process may handle that related object appropriately. This case can usually only arise if an object is removed from one :class:`.Session` -- 2.47.3