Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314109 - org.eclipse.jst.ejb.ui.tests needs an about.html file
Summary: org.eclipse.jst.ejb.ui.tests needs an about.html file
Status: RESOLVED FIXED
Alias: None
Product: WTP EJB Tools
Classification: WebTools
Component: jst.ejb (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P1 normal (vote)
Target Milestone: 3.2 RC3   Edit
Assignee: Kaloyan Raev CLA
QA Contact: Kaloyan Raev CLA
URL:
Whiteboard: PMC_approved
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-24 08:55 EDT by David Williams CLA
Modified: 2010-05-27 17:36 EDT (History)
0 users

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

Attachments
patch (21.58 KB, patch)
2010-05-26 08:05 EDT, Kaloyan Raev CLA 		no flags Details | Diff
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 David Williams CLA 2010-05-24 08:55:28 EDT 
While adding that, how about adding a .qualifier, and externalizing strings.
Comment 1 Kaloyan Raev CLA 2010-05-26 08:05:43 EDT 
Created attachment 169974 [details]
patch

- about.html added
- qualifier added
- string externalized
Comment 2 Kaloyan Raev CLA 2010-05-26 08:08:32 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. 

about.html is required for all plugins. This plugin has been introduced in Helios, but the about.html was forgotten.

    * 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? 

The test still runs when executed from the workbench. 

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

about.html is added. As well as some other correction suggested by David.

    * What is the risk associated with this fix? 

Low.
Comment 3 Kaloyan Raev CLA 2010-05-27 06:54:36 EDT 
Committed and released to HEAD