Summary: | Axis2: Support import of AAR | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [WebTools] WTP Webservices | Reporter: | Kathy Chan <kathy> | ||||||||||
Component: | jst.ws | Assignee: | Saminda Wijeratne <samindaw> | ||||||||||
Status: | NEW --- | QA Contact: | Keith Chong <keith.chong.ca> | ||||||||||
Severity: | enhancement | ||||||||||||
Priority: | P2 | CC: | keith.chong.ca, project-eclipse | ||||||||||
Version: | 2.0 | Keywords: | helpwanted | ||||||||||
Target Milestone: | Future | ||||||||||||
Hardware: | PC | ||||||||||||
OS: | Windows XP | ||||||||||||
Whiteboard: | Axis2 | ||||||||||||
Attachments: |
|
Description
Kathy Chan
2006-12-22 10:42:04 EST
Not contained in M5. Deferred to M6. Could not make it into WTP 2.0 M6. Deferred out of WTP 2.0. Targetting WTP 3.0. Setting target field to future since all WTP 3.0 RFEs needs to be in by M6 which had already been declared. Please put a more accurate target when these RFEs are in plan for a particular release. Created attachment 127637 [details]
icon for patch for 168937
This icon is created in WSO2 for the purpose of representing axis2 archives. I
got the OK for using this icon in eclipse from Samisa. Therefore we are
herewith giving permission to use this icon to eclipse wtp axis2 project under
the eclipse license.
Just extract the icons.zip file to the
org.eclipse.jst.ws.axis2.consumption.ui project folder.
Created attachment 127638 [details]
patch for 168937
This patch contains the updated code for the importing functionality including the
code for specifying the icon.
Created attachment 128248 [details]
Null Pointer
If you do an import without first selecting a project, you will get the null pointer.
Hi Saminda, I think the Axis2 Service Archive Importer wizard page could have better fit and finish. ie. when you resize the dialog width-wise, you'll notice the spacing between the labels and the text field. The left part of the text fields should still remain fixed against the right side of the label. Also, note that if you press Back, it goes to an empty page. It should go back to the regular Import page. Please externalize the strings. Hi Saminda, Any outlook on updating your patch with fixes to address the issues I've found? I think this and the sibling bug 168938 should be checked in at the same time. M6 is going to be declared this week, so I think if we are going to get these fixes in, we should defer this* until M7 (earlier the better) and seek PMC approval for these changes. What do you think? *if possible, pull out all the strings into the properties file and we can check these string changes in for M6, tomorrow (Tuesday). At least we will have the strings available in M7. Deferring to 3.1 M7. Hi Keith, I've being trying to figure out why a blank page appears when back button is pressed. But I couldn't find the reason. I couldn't find a decent example of an import/export wizard to compare my work with to check where I'm going wrong. Being struggling with this for a couple of days now. Any ideas?? Thanx, Saminda Hi Saminda, let's revisit this enhancement for 3.2. (In reply to comment #11) > Hi Keith, > I've being trying to figure out why a blank page appears when back button is > pressed. But I couldn't find the reason. I couldn't find a decent example of an > import/export wizard to compare my work with to check where I'm going wrong. > Being struggling with this for a couple of days now. Any ideas?? > Thanx, > Saminda Saminda, sorry, I don't have the time to look at this. I will move designate this and its companion bug (export AAR) as helpwanted. Please modify if necessary. Created attachment 162242 [details] patch for 168937 Fixed the NPE and the problem of showing a blank page when back is clicked. The reason was that I had set the parent composite as the control instead of the container in the wizard page createControl method. This patch shares code with bug 168938. Please verify and commit. Saminda |