[jira] [Updated] (SLING-5721) Extend Sling Models to allow mapping of ValueMap values to custom types

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5721?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Justin Edelson updated SLING-5721: -- Attachment: SLING-5721.patch here's my proposed change to do this automatically, including a uni

[jira] [Commented] (SLING-5726) Allow ValueMapInjector and ResourcePathInjector to act directly on SlingHttpServletRequest

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5726?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293686#comment-15293686 ] Justin Edelson commented on SLING-5726: --- (Trying to move this conversation to JIRA n

Re: svn commit: r1743648 - in /sling: site/trunk/content/documentation/bundles/ trunk/bundles/extensions/models/impl/src/main/java/org/apache/sling/models/impl/injectors/ trunk/bundles/extensions/mode

2016-05-20 Thread Justin Edelson
Hi Georg, I'm entirely supportive of the notion of using models in multiple contexts. My concern here is one of separation of concerns. AbstractInjector is *not* API, it is an internal helper. Putting additional expectations on Injectors just seems wrong to me. What we could do as an alternative w

[jira] [Commented] (SLING-5721) Extend Sling Models to allow mapping of ValueMap values to custom types

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293679#comment-15293679 ] Justin Edelson commented on SLING-5721: --- Thinking about this some more, I'm not sure

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293650#comment-15293650 ] Justin Edelson commented on SLING-5739: --- bq. Generally, I think it will almost alway

[jira] [Comment Edited] (SLING-5719) Missing isLive() test before closing ResourceResolver

2016-05-20 Thread Jens Lauterbach (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5719?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293623#comment-15293623 ] Jens Lauterbach edited comment on SLING-5719 at 5/20/16 4:12 PM: ---

[jira] [Commented] (SLING-5719) Missing isLive() test before closing ResourceResolver

2016-05-20 Thread Jens Lauterbach (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5719?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293623#comment-15293623 ] Jens Lauterbach commented on SLING-5719: You can get a {{ResourceResolverFactory}}

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293618#comment-15293618 ] Justin Edelson commented on SLING-5739: --- nevermind. I figured out my error. I'm not

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293593#comment-15293593 ] Justin Edelson commented on SLING-5739: --- This doesn't work for me. Take a look at th

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Georg Henzler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293565#comment-15293565 ] Georg Henzler commented on SLING-5739: -- {quote} this is actually about injecting a pr

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293512#comment-15293512 ] Konrad Windszus commented on SLING-5739: Please consider my example from above. At

[jira] [Issue Comment Deleted] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus updated SLING-5739: --- Comment: was deleted (was: This is different than the other Via then. For example when using V

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293500#comment-15293500 ] Konrad Windszus commented on SLING-5739: Actually you can already achieve that via

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293492#comment-15293492 ] Konrad Windszus commented on SLING-5739: This is different than the other Via then

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293462#comment-15293462 ] Justin Edelson commented on SLING-5739: --- [~kwin] This is orthogonal to `@Named`. `@N

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293460#comment-15293460 ] Justin Edelson commented on SLING-5739: --- In terms of other `@Via` providers, one mor

[jira] [Comment Edited] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293457#comment-15293457 ] Justin Edelson edited comment on SLING-5739 at 5/20/16 2:43 PM:

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Justin Edelson (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293457#comment-15293457 ] Justin Edelson commented on SLING-5739: --- [~henzlerg] this is actually about injectin

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293408#comment-15293408 ] Konrad Windszus commented on SLING-5739: For me introducing more options for {{Via

[jira] [Commented] (SLING-5739) [Sling Models] Allow for extensible @Via providers

2016-05-20 Thread Georg Henzler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293362#comment-15293362 ] Georg Henzler commented on SLING-5739: -- I think the use case from this ticket already

[jira] [Updated] (SLING-5702) SlingFileUploadHandler creates unnecessary chunk when upload part is empty

2016-05-20 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5702?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler updated SLING-5702: Fix Version/s: (was: Servlets Post 2.3.12) > SlingFileUploadHandler creates unnecessary

[jira] [Resolved] (SLING-5702) SlingFileUploadHandler creates unnecessary chunk when upload part is empty

2016-05-20 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5702?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler resolved SLING-5702. - Resolution: Cannot Reproduce > SlingFileUploadHandler creates unnecessary chunk when uploa

[jira] [Commented] (SLING-5702) SlingFileUploadHandler creates unnecessary chunk when upload part is empty

2016-05-20 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5702?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293307#comment-15293307 ] Carsten Ziegeler commented on SLING-5702: - I can't see how that could happen. I th

[jira] [Resolved] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus resolved SLING-5740. Resolution: Fixed Fix Version/s: i18n 2.4.8 Applied the fix together with some test c

[jira] [Commented] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293181#comment-15293181 ] ASF GitHub Bot commented on SLING-5740: --- Github user asfgit closed the pull request

[GitHub] sling pull request: SLING-5740 extracting country from locale Stri...

2016-05-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/sling/pull/142 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabl

[jira] [Comment Edited] (SLING-5355) Create service users and ACLs from the provisioning model

2016-05-20 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293119#comment-15293119 ] Bertrand Delacretaz edited comment on SLING-5355 at 5/20/16 10:33 AM: --

[jira] [Commented] (SLING-5449) Content repository initialization language

2016-05-20 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293137#comment-15293137 ] Bertrand Delacretaz commented on SLING-5449: The following enhancements to the

[jira] [Assigned] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Konrad Windszus reassigned SLING-5740: -- Assignee: Konrad Windszus > [i18n] extracting country from locale String is incorrect i

[jira] [Commented] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293131#comment-15293131 ] Konrad Windszus commented on SLING-5740: Sorry, I got that only right after having

[jira] [Commented] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Konrad Windszus (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293130#comment-15293130 ] Konrad Windszus commented on SLING-5740: The fix is not correct, but there is defi

[jira] [Commented] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Ankit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293125#comment-15293125 ] Ankit Agarwal commented on SLING-5740: -- pull req https://github.com/apache/sling/pull

[jira] [Commented] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293122#comment-15293122 ] ASF GitHub Bot commented on SLING-5740: --- GitHub user anagarwa opened a pull request:

[jira] [Comment Edited] (SLING-5355) Create service users and ACLs from the provisioning model

2016-05-20 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293119#comment-15293119 ] Bertrand Delacretaz edited comment on SLING-5355 at 5/20/16 10:21 AM: --

[GitHub] sling pull request: SLING-5740 extracting country from locale Stri...

2016-05-20 Thread anagarwa
GitHub user anagarwa opened a pull request: https://github.com/apache/sling/pull/142 SLING-5740 extracting country from locale String is incorrect in JcrR… …esourceBundleProvider You can merge this pull request into a Git repository by running: $ git pull https://github.com

[jira] [Commented] (SLING-5449) Content repository initialization language

2016-05-20 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293121#comment-15293121 ] Bertrand Delacretaz commented on SLING-5449: This is now used in SLING-5355 an

[jira] [Commented] (SLING-5355) Create service users and ACLs from the provisioning model

2016-05-20 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15293119#comment-15293119 ] Bertrand Delacretaz commented on SLING-5355: At revision 1744682 I have implem

[jira] [Updated] (SLING-5740) [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Ankit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ankit Agarwal updated SLING-5740: - Summary: [i18n] extracting country from locale String is incorrect in JcrResourceBundleProvider (

[jira] [Updated] (SLING-5740) [i18n] extracting country from String is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Ankit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ankit Agarwal updated SLING-5740: - Summary: [i18n] extracting country from String is incorrect in JcrResourceBundleProvider (was: [i

[jira] [Updated] (SLING-5740) [i18n] extracting country code is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Ankit Agarwal (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ankit Agarwal updated SLING-5740: - Description: while extracting Locale from string, country code as compared to lang https://githu

[jira] [Created] (SLING-5740) [i18n] extracting country code is incorrect in JcrResourceBundleProvider

2016-05-20 Thread Ankit Agarwal (JIRA)
Ankit Agarwal created SLING-5740: Summary: [i18n] extracting country code is incorrect in JcrResourceBundleProvider Key: SLING-5740 URL: https://issues.apache.org/jira/browse/SLING-5740 Project: Sling

[jira] [Commented] (SLING-5666) Unscheduling a job should remove corresponding node

2016-05-20 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15292961#comment-15292961 ] Carsten Ziegeler commented on SLING-5666: - [~obrist] I think the code is now corre

[jira] [Resolved] (SLING-5666) Unscheduling a job should remove corresponding node

2016-05-20 Thread Carsten Ziegeler (JIRA)
[ https://issues.apache.org/jira/browse/SLING-5666?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Carsten Ziegeler resolved SLING-5666. - Resolution: Fixed > Unscheduling a job should remove corresponding node >