This is an automated email from the ASF dual-hosted git repository. ahuber pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/isis.git
commit 34a6180e7b5e93a34c7061b8574e127f78f2a4e0 Author: Andi Huber <ahu...@apache.org> AuthorDate: Thu Jun 30 10:44:46 2022 +0200 ISIS-3084: just comments --- .../core/config/progmodel/ProgrammingModelConstants.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/core/config/src/main/java/org/apache/isis/core/config/progmodel/ProgrammingModelConstants.java b/core/config/src/main/java/org/apache/isis/core/config/progmodel/ProgrammingModelConstants.java index ea17f20629..a161a9ad7f 100644 --- a/core/config/src/main/java/org/apache/isis/core/config/progmodel/ProgrammingModelConstants.java +++ b/core/config/src/main/java/org/apache/isis/core/config/progmodel/ProgrammingModelConstants.java @@ -508,15 +508,17 @@ public final class ProgrammingModelConstants { } - //TODO needs an update to reflect Java 7->11 Language changes + //TODO perhaps needs an update to reflect Java 7->11 Language changes @RequiredArgsConstructor public static enum WrapperFactoryProxy { COLLECTION( + // intercepted ... List.of( getMethod(Collection.class, "contains", Object.class), getMethod(Collection.class, "size"), getMethod(Collection.class, "isEmpty") ), + // vetoed ... List.of( getMethod(Collection.class, "add", Object.class), getMethod(Collection.class, "remove", Object.class), @@ -526,24 +528,28 @@ public final class ProgrammingModelConstants { getMethod(Collection.class, "clear") )), LIST( + // intercepted ... _Lists.concat( - WrapperFactoryProxy.COLLECTION.intercepted, + COLLECTION.intercepted, List.of( getMethod(List.class, "get", int.class) ) ), + // vetoed ... _Lists.concat( - WrapperFactoryProxy.COLLECTION.vetoed, + COLLECTION.vetoed, List.of( ) )), MAP( + // intercepted ... List.of( getMethod(Map.class, "containsKey", Object.class), getMethod(Map.class, "containsValue", Object.class), getMethod(Map.class, "size"), getMethod(Map.class, "isEmpty") ), + // vetoed ... List.of( getMethod(Map.class, "put", Object.class, Object.class), getMethod(Map.class, "remove", Object.class),