[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-20 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14038529#comment-14038529
 ] 

ASF GitHub Bot commented on SLING-3651:
---

Github user kwin closed the pull request at:

https://github.com/apache/sling/pull/19


 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus
Assignee: Stefan Egli
 Fix For: Sling Eclipse IDE 1.0.0


 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-16 Thread Konrad Windszus (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14032180#comment-14032180
 ] 

Konrad Windszus commented on SLING-3651:


Yes, the changes are all included in the pull request. Would you consider 
integrating that in Version 1.0.0?

 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-16 Thread Stefan Egli (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14032245#comment-14032245
 ] 

Stefan Egli commented on SLING-3651:


Yes, I think this should go into 1.0.0. I'll look at it next.

 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus
Assignee: Stefan Egli

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-10 Thread Stefan Egli (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14026316#comment-14026316
 ] 

Stefan Egli commented on SLING-3651:


Hi [~kwin], thanks!
bq. I also improved the path handling
Does 'https://github.com/apache/sling/pull/19' include this change already?

 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-06 Thread Konrad Windszus (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14019714#comment-14019714
 ] 

Konrad Windszus commented on SLING-3651:


Also I would like if the ProjectHelper.getInferredContentProjectContentRoot() 
would only be taken as the default for the dialog (asking for the JCR-Root). 
But it should be possible to convert projects, even if the JCR-Root is outside 
the checked folders. In some project the jcr_root is at /src/main/jcr_root.

 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-06 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14019829#comment-14019829
 ] 

ASF GitHub Bot commented on SLING-3651:
---

GitHub user kwin opened a pull request:

https://github.com/apache/sling/pull/19

SLING-3651, improved error message for conversion to content project

always enable the conversion action (to be able to see the error
message)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kwin/sling SLING-3651-improve-conversions

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/sling/pull/19.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #19


commit 57cebdb13ffb0d380bfdaddece2219d0d4d680ee
Author: Konrad Windszus konrad.winds...@netcentric.biz
Date:   2014-06-06T13:15:16Z

SLING-3651, improved error message for conversion to content project,
always enable the conversion action (to be able to see the error
message)




 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-06 Thread Konrad Windszus (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14020011#comment-14020011
 ] 

Konrad Windszus commented on SLING-3651:


I also improved the path handling (to be platform independent and even support 
Windows and backslashes :-)). The Conversion To Bundle Wizard will now display 
a proper tree view to allow me to easily select the JCR root (instead of a 
simple input field) with the correct validation and error messages in place. 

 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (SLING-3651) Improve Conversion To Bundle and Content Projects

2014-06-06 Thread Konrad Windszus (JIRA)

[ 
https://issues.apache.org/jira/browse/SLING-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14020022#comment-14020022
 ] 

Konrad Windszus commented on SLING-3651:


What would be good to do as well, is to reuse the same dialog with the tree 
view for the Sling Property Page (because it only shows containers below the 
project and gives out validation information). [~rombert] Maybe you can do 
that. Otherwise we can defer that part to a future version.

 Improve Conversion To Bundle and Content Projects
 -

 Key: SLING-3651
 URL: https://issues.apache.org/jira/browse/SLING-3651
 Project: Sling
  Issue Type: Bug
  Components: IDE
Affects Versions: Sling Eclipse IDE 1.0.0
Reporter: Konrad Windszus

 Currently the Convert To Bundle Project is always enabled and I can even 
 add the bundle facet to a maven project which is of packaging 
 content-package
 The Convert To Content Project... is only enabled under very specific 
 conditions. Currently those conditions are neither described in 
 https://cwiki.apache.org/confluence/display/SLING/Sling+IDE+tooling+User+Guide
  nor is there some warning/trace messages available. 
 I would suggest the following:
 Always enable both conversion actions. Display a warning message with a very 
 descriptive warning in case the conversion cannot be performed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)