[SLING-11601] [Scripting] Use the LazyBindings instead of SimpleBindings

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: Sling Models Impl 1.4.0
• Fix Version/s: Scripting API 2.2.2, Models Implementation 1.5.4, Scripting HTL Engine 1.4.22-1.4.0, Testing Sling Mock 3.4.4, Scripting HTL JS Use Provider 1.2.10
• Component/s: Sling Models
• Assignee: Joerg Hoh
• Reporter: Joerg Hoh
• Created: 27/Sep/22 10:11
• Updated: 18/Apr/23 10:56
• Resolved: 27/Oct/22 16:11

Description

At various modules Sling Scripting (and Sling Models) code still uses a SimpleBindings implementation of the Bindings, while it could use LazyBindings which are more efficient in case not all provided bindings are required.
While the scripting itself does not necessarily use a LazyBindings.Supplier, other BindingValueProviders can be invoked on them, which do support the LazyBindings; and in that case there can be a benefit.

Requires a bump of the dependency to Sling API though (at least 2.21.0 is required)

Issue Links

• is related to: SLING-11832 Embed latest version of org.apache.sling.scripting.core to make use of LazyBindings