This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 226773 - provide ui to select license template
Summary: provide ui to select license template
Status: RESOLVED FIXED
Alias: None
Product: projects
Classification: Unclassified
Component: Ant Project (show other bugs)
Version: 7.3
Hardware: PC Mac OS X
: P3 normal (vote)
Assignee: Milos Kleint
URL:
Keywords: API, API_REVIEW_FAST
Depends on: 193981
Blocks: 204221 227362 227364 227365
  Show dependency tree
 
Reported: 2013-02-27 11:45 UTC by Milos Kleint
Modified: 2013-06-07 05:11 UTC (History)
2 users (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments
api module changes, includes api and implementation (36.76 KB, patch)
2013-03-06 16:51 UTC, Milos Kleint
Details | Diff
j2se project implementation (11.68 KB, patch)
2013-03-06 16:52 UTC, Milos Kleint
Details | Diff
maven projects implementation (8.52 KB, patch)
2013-03-06 16:52 UTC, Milos Kleint
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Milos Kleint 2013-02-27 11:45:49 UTC
see issue 193981, ant projects now allow setting project.licensePath property in nbproject/project.properties that points to a file in project space (absolute path)

We will need some UI in Project Properties that would allow
1. setting a license from the current list in IDE
2. allow setting a reference to file in project space
3. allow creating a license template file in project space.
Comment 1 Milos Kleint 2013-03-06 16:51:21 UTC
Created attachment 132283 [details]
api module changes, includes api and implementation
Comment 2 Milos Kleint 2013-03-06 16:52:31 UTC
Created attachment 132284 [details]
j2se project implementation
Comment 3 Milos Kleint 2013-03-06 16:52:52 UTC
Created attachment 132285 [details]
maven projects implementation
Comment 4 Milos Kleint 2013-03-06 16:54:09 UTC
please review this new api to create license header editing customizer panel for ant and maven projects (or anything else depending on projects.ant)
Comment 5 Martin Fousek 2013-03-07 08:30:12 UTC
API looks well to me.

few details:
MF01 - first patch is uncompilable now: DefaultComboBoxModel<GlobalItem>
MF02 - UI panel left, top paddings are not consistent with the rest of project customizer UI panels
MF03 - no Common Annotation usage

j2se client:
MF04 - J2SECompositePanelProvider#setProjectLicenseLocation handles wrong the <code>null<code> value

BTW, relative project.licensePath didn't work to me - issue #227139.
Comment 6 Milos Kleint 2013-03-12 08:38:02 UTC
thanks for the review, will implement MF01-04 and integrate shortly
Comment 7 Milos Kleint 2013-03-12 14:39:56 UTC
http://hg.netbeans.org/core-main/rev/83544a84772b
Comment 8 Quality Engineering 2013-03-14 02:08:21 UTC
Integrated into 'main-golden', will be available in build *201303132300* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/83544a84772b
User: Milos Kleint <mkleint@netbeans.org>
Log: #226773, #226774 provide ui for selecting license header panel in customizers for maven and j2se projects
Comment 9 David Konecny 2013-06-07 02:30:09 UTC
Milos, why was org.netbeans.spi.project.support.ant.ui.CustomizerUtilities put into ant.project module instead of projectuiapi when it is used by all project types - Ant, Maven, HTML5, PHP?
Comment 10 Milos Kleint 2013-06-07 05:04:10 UTC
(In reply to comment #9)
> Milos, why was org.netbeans.spi.project.support.ant.ui.CustomizerUtilities put
> into ant.project module instead of projectuiapi when it is used by all project
> types - Ant, Maven, HTML5, PHP?


well, likely because noone objected at apireview time. :)

I've just picked the lowest common denominator I could think of and ant support is effectively used by maven because of higher level dependencies. Do you have a project type case that actually has to add the ant project dependency?
Comment 11 David Konecny 2013-06-07 05:11:32 UTC
No, I do not have one. HTML5 project type is using Ant - in order to build PhoneGap applications Ant script is used internally. But I'm surprised to hear that Maven is using Ant. It's not a big deal - I was just surprised.