[GitHub] neilcsmith-net commented on issue #425: Netbeans 352

2018-02-26 Thread GitBox
neilcsmith-net commented on issue #425: Netbeans 352
URL: 
https://github.com/apache/incubator-netbeans/pull/425#issuecomment-368455829
 
 
   @JaroslavTulach fair enough.  I've obviously made a lousy case for why this 
is impossible to fix _inside_ the platform, sorry! :smile:
   
   I've opened an issue in JIRA, linked to the original issue - not sure it's 
good discussing this on a PR.  Maybe we should close this and find consensus 
there?  Issues related to this do crop up now and again, but it would be good 
to have a solution that you can be in favour of!  And also one that solves 
@SirIntellegence problem.
   
   https://issues.apache.org/jira/browse/NETBEANS-427


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists



[GitHub] neilcsmith-net commented on issue #425: Netbeans 352

2018-02-23 Thread GitBox
neilcsmith-net commented on issue #425: Netbeans 352
URL: 
https://github.com/apache/incubator-netbeans/pull/425#issuecomment-368035513
 
 
   To use pluggable formatters we'd still have to be able to wrap or override 
`NbBundle.getMessage()` to use our own `MessageFormat` to achieve this?
   
   It's not a proposal to break the idea of referentially transparent you're 
talking about then - it's about being able to override / wrap the default 
behaviour of `NbBundle.getMessage()`.  That _could_ be abused, but the proposal 
is meant to be that you still get the same output for any distinct set of input 
arguments.
   
   At the moment there is an assumption that a module is providing all / the 
right inputs you need to construct the output message you want, but this 
doesn't always hold in a platform application (it probably does for 
translations).  The OP's problem is one, but there are lots of times you might 
want to calculate something differently eg. turning an `X of Y files` message 
into an `X files remaining` message, etc.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists



[GitHub] neilcsmith-net commented on issue #425: Netbeans 352

2018-02-21 Thread GitBox
neilcsmith-net commented on issue #425: Netbeans 352
URL: 
https://github.com/apache/incubator-netbeans/pull/425#issuecomment-367251507
 
 
   Surely `NbBundle.getMessage()` isn't referentially transparent already?  At 
least with respect to locale and branding token, otherwise there wouldn't be a 
need for it.  Filter might have been a bad choice of word (think 
FilterInputStream) - but, yes, really I'm wanting a way to dynamically (well, 
at run time) transform the input arguments!
   
   Taking a step back, branding cannot achieve what the OP wants to achieve 
because it's static, and here  there's a need to transform the input arguments 
into the message via code.  I've come across, and seen other people with, 
similar problems in various different areas of the platform, and sometimes 
hacked around them (including exactly what the OP is doing for this).  I was 
trying to come up with the least intrusive way we could fix it in one place, 
rather than PR's all over the platform and / or requiring people to rewrite big 
chunks of it.
   
   They all come down to needing to change the input arguments to (or wrap 
entirely) `NbBundle.getMessage()`, _from code_.  Am I right in thinking there 
is no way to do that currently?  Do you have ideas on a better way to achieve 
this?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists



[GitHub] neilcsmith-net commented on issue #425: Netbeans 352

2018-02-20 Thread GitBox
neilcsmith-net commented on issue #425: Netbeans 352
URL: 
https://github.com/apache/incubator-netbeans/pull/425#issuecomment-367038068
 
 
   @JaroslavTulach the actual issue here cannot be handled by branding as far 
as I can see - there needs to be a way to filter / change the arguments passed 
into `NbBundle.getMessage()` to do this.  As I mentioned on the first PR for 
this, I'm -1 to adding a new API to try and fix this one issue, because it's 
not the only place I've seen this problem.  I did suggest we provide a way to 
filter input arguments - 
https://github.com/apache/incubator-netbeans/pull/393#issuecomment-362662452  
What would be your thoughts on trying to do that instead?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

-
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists