Skip to content

Commit

Permalink
property-hooks.xml Replace 'all' with 'non-static', as not all + typos (
Browse files Browse the repository at this point in the history
#4116)

* Update property-hooks.xml Replace 'all' with 'non-static', as not all

The word `all` seems to deceive the reader a little :)

How about adding some specifics about which properties are allowed to declare hooks for?

* Update property-hooks.xml remove the dot cahr

* Update property-hooks.xml fix typo

* Update property-hooks.xml Remove extra parentheses

* property-hooks.xml move the period back inside the quotation marks,

* property-hooks.xml Typed and non-types

Specify that hooks are available for both typed and untyped properties
  • Loading branch information
mmalferov authored Nov 24, 2024
1 parent c35ad60 commit b34210e
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions language/oop5/property-hooks.xml
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,9 @@
</listitem>
</orderedlist>
<simpara>
There are two hooks available on all properties: <literal>get</literal> and <literal>set</literal>.
There are two hooks available on non-static properties: <literal>get</literal> and <literal>set</literal>.
They allow overriding the read and write behavior of a property, respectively.
Hooks are available for both typed and untyped properties.
</simpara>
<simpara>
A property may be "backed" or "virtual".
Expand Down Expand Up @@ -228,7 +229,7 @@ class Example
</example>
<simpara>
For virtual properties, if a hook is omitted then that operation does
not exist and trying to use it wil produce an error.
not exist and trying to use it will produce an error.
Virtual properties take up no memory space in an object.
Virtual properties are suited for "derived" properties,
such as those that are the combination of two other properties.
Expand Down

0 comments on commit b34210e

Please sign in to comment.