Bug 339572 - [NLS] Superfluous "(Java EE 6)" in the EJB Timer wizard action
Summary: [NLS] Superfluous "(Java EE 6)" in the EJB Timer wizard action
Status: RESOLVED FIXED
Alias: None
Product: WTP EJB Tools
Classification: WebTools
Component: jst.ejb (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows Vista
: P3 normal (vote)
Target Milestone: 3.3 M7   Edit
Assignee: ludo CLA
QA Contact: Kaloyan Raev CLA
URL:
Whiteboard: PMC_approved
Keywords: PII
Depends on:
Blocks:
 
Reported: 2011-03-10 14:11 EST by Kaloyan Raev CLA
Modified: 2011-04-28 22:10 EDT (History)
1 user (show)

See Also:
kaloyan: pmc_approved? (david_williams)
kaloyan: pmc_approved? (raghunathan.srinivasan)
kaloyan: pmc_approved? (naci.dai)
kaloyan: pmc_approved? (deboer)
kaloyan: pmc_approved? (neil.hauge)
kaloyan: pmc_approved? (kaloyan)
cbridgha: pmc_approved+
kaloyan: review+

Attachments
proposed patch (773 bytes, text/plain)
2011-04-21 11:38 EDT, ludo CLA 		kaloyan: iplog+
Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kaloyan Raev CLA 2011-03-10 14:11:17 EST 
The New EJB Timer wizard action is labeled as "EJB Timer (Java EE 6)". IMHO, this "(Java EE 6)" is not necessary, because EJB Timers are introduced in Java EE 6 and they are not available in previous versions of the spec. 

I suggest to remove it to keep the labels of the EJB artifact actions consistent.
Comment 1 Kaloyan Raev CLA 2011-04-21 11:10:06 EDT 
Ludo, since you are the contributor of this feature, could you work on proposing a patch for this issue?
Comment 2 ludo CLA 2011-04-21 11:38:30 EDT 
Created attachment 193853 [details]
proposed patch

agree. See the patch.
Comment 3 Kaloyan Raev CLA 2011-04-26 04:17:29 EDT 
    Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. 

The support for EJB Timers has been introduced with M6. This patch fixes a small UI inconsistency. It's good to fix it now - before the first release of WTP that introduces this new feature. 

    Is there a work-around? If so, why do you believe the work-around is insufficient? 

No.

    How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added? 

Manually tested. 

    Give a brief technical overview. Who has reviewed this fix? 

This is a simple change in a text label. 

    What is the risk associated with this fix? 

No risk.
Comment 4 Chuck Bridgham CLA 2011-04-26 10:29:30 EDT 
I'm ok with this
Comment 5 Kaloyan Raev CLA 2011-04-26 10:54:51 EDT 
Comitted and released to HEAD