gmane.comp.cms.magnolia.devel

JIRA (on behalf of Fabrizio Giustina | 16 Jul 21:46

[magnolia-dev] [JIRA] Created: (MAGNOLIA-2273) Make MessagesManager easier to extend/replace

Make MessagesManager easier to extend/replace
---------------------------------------------

                 Key: MAGNOLIA-2273
                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2273
             Project: Magnolia
          Issue Type: Improvement
    Affects Versions: 3.6 RC1
            Reporter: Fabrizio Giustina
            Assignee: Fabrizio Giustina
             Fix For: 3.6

The current implementation ca't in any way be integrated/replaced in order to manage translations in a
different way (for example saving them into the magnolia repo)... we should add a bit of flexibility

--

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

headers

JIRA (on behalf of Fabrizio Giustina | 16 Jul 21:58

[magnolia-dev] [JIRA] Commented: (MAGNOLIA-2270) Better logs and avoid NPEs when magnolia is started from an unexpanded war


    [
http://jira.magnolia.info/browse/MAGNOLIA-2270?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17161#action_17161
] 

Fabrizio Giustina commented on MAGNOLIA-2270:
---------------------------------------------

> I would prefer not to make changes to this kind of very central classes while being in RC.

the only change is the exception message, there is nothing else changed here... we just tried to run it on
tomcat6 from an unexpanded war and we saw that it crashes since
context.getRealPath(StringUtils.EMPTY) returns a null, nothing more.

> Better logs and avoid NPEs when magnolia is started from an unexpanded war
> --------------------------------------------------------------------------
>
>                 Key: MAGNOLIA-2270
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2270
>             Project: Magnolia
>          Issue Type: Improvement
>    Affects Versions: 3.6 RC1
>            Reporter: Fabrizio Giustina
>            Assignee: Fabrizio Giustina
>             Fix For: 3.6
>
>
> Magnolia doesn't work with if started from an unexpanded war, as we know, but we also should give a better output.
> 3.6-rc1 dies with an unhelpful NPE in MgnlServletContextListener

(Continue reading)

headers

JIRA (on behalf of Gregory Joseph | 16 Jul 22:20

[magnolia-dev] [JIRA] Reopened: (MAGNOLIA-2270) Better logs and avoid NPEs when magnolia is started from an unexpanded war


     [
http://jira.magnolia.info/browse/MAGNOLIA-2270?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gregory Joseph reopened MAGNOLIA-2270:
--------------------------------------

Changing the exception message (and mostly avoiding the npe when generating it) is a good thing, but i'd
like to get rid of the false statement that was introduced in it. thanks.

> Better logs and avoid NPEs when magnolia is started from an unexpanded war
> --------------------------------------------------------------------------
>
>                 Key: MAGNOLIA-2270
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2270
>             Project: Magnolia
>          Issue Type: Improvement
>    Affects Versions: 3.6 RC1
>            Reporter: Fabrizio Giustina
>            Assignee: Fabrizio Giustina
>             Fix For: 3.6
>
>
> Magnolia doesn't work with if started from an unexpanded war, as we know, but we also should give a better output.
> 3.6-rc1 dies with an unhelpful NPE in MgnlServletContextListener

--

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.magnolia.info/secure/Administrators.jspa

(Continue reading)