[jira] [Commented] (ISIS-1342) The metamodel validation error page doesn't reliably render itself if there are errors.

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ISIS-1342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15211506#comment-15211506 ] 

ASF subversion and git services commented on ISIS-1342:
-------------------------------------------------------

Commit 8bf7fd7f04049e3632454c34e1ca329df1a936f7 in isis's branch refs/heads/master from [~danhaywood]
[ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=8bf7fd7 ]

ISIS-1342: IsisSystem no longer throws MetaModelException, instead pushes it onto the IsisContext.  Its callers (IsisWicketApplication, or IsisSessionFilter - for RO - or CukeGlueAbstract, or IsisSystemForTest) then expose the error appropriately, accessing from IsisContext.

Additional details:
- IsisSystem no longer initializes the AuthenticationManager, AuthorizationManager or PersistenceSessionFactory if there was a metamodel validation, to minimize impact on runtime.
- IsisSystemForTest will fail with a minimal error for any subsequent tests
- IsisContext now as new metamodeInvalidException field.  This is static rather than held on the IsisContext instance as a threadlocal; that's because IsisSystemForTest actually calls IsisContext.testReset() to discard the IsisContext singleton.
- IsisSessionFilter (which sits in front of RO viewer) now always checks that the metamodel is vaild.  It does this simply by querying the IsisContext.  (If WebAppBootstrapper was used to bootstrap, rather than wicket viewer, then IsisSystem.init() is still called so no diff in end result)
- ScenarioException simply does a fail() if no instance available on threadlocal.
- CukeGlueAbstract similarly has simple guards for if ScenarioException available, to reduce amount of noise when problem is metamodel invalid
- formatting changes only for IsisWicketApplication
- WebRequestCycleForIsis looks for validation errors first-off in its onException handler, if any are found then immediately redirects to the MmvErrorPage.  This seems to work fine for always rendering MMV page.

Improvements:
- IsisMojoAbstract keeps the IsisSystem initialized while it executes.


> The metamodel validation error page doesn't reliably render itself if there are errors.
> ---------------------------------------------------------------------------------------
>
>                 Key: ISIS-1342
>                 URL: https://issues.apache.org/jira/browse/ISIS-1342
>             Project: Isis
>          Issue Type: Bug
>          Components: Core
>            Reporter: Dan Haywood
>            Assignee: Dan Haywood
>            Priority: Minor
>             Fix For: 1.12.0
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)