Community
Participate
Working Groups
Build ID: WTP 201 This is coming in response to a defect in adopter product where users cannot find a way to choose existing JSP. Steps To Reproduce: Create servlet wizard should say 'Use existing Servlet class or JSP' It currently says Create servlet wizard should say 'Use existing Servlet class' and has a browse button, but if you click that you can select a servlet class or JSP and by spec you are allowed to choose either. simple string value change required, patch attached More information:
Created attachment 76349 [details] cvs patch
Raj, I don't think this meets the criteria for 2.0.1... can we wait until 3.0?
This change is only a UI change, and we don't generally do UI or string changes in a maintenance release.
Yes, Its a UI change and normally I would not be asking for this change, but this is really a glaring error, I dont know how it was not caught earlier by other users in WTP. As I mentioned earlier, because the wizard says 'Use existing Servlet class' there is no way for users to know HOW they can add a JSP. And this was brought to our attention in an adopter product where it was considered to be a regression because Servlet wizards (previously versions of the adopter product) would display the option to choose a JSP more clearly (its not clear here).
I discussed this with Chuck- he agrees with Nitin and myself. This change will go into the 3.0 stream when we branch. Also, renaming the constant that is used to retrieve the string is not a good practice- the name of the constant may not be 100% correct now, but this sort of practice can lead to all sorts of problems. (You should just change the contents of the string in the .properties file.) Can you simplify this patch and reattach?
Created attachment 76777 [details] patch that has only value changes Patch with only the property value being changed. Regarding the previous comment, about renaming the constant,the reason I did it was 1. its an internal constant, so nobody should be using it anyway. 2. when name of the constant is 'use servlet' and value is 'use servlet or jsp' why not change the name of the constant to be more accurate. even though I agree just changing the value is more non-visible change and since WTP 201 is a maintenance release, so I have attached the patch.
Committed to HEAD
mass change to add 'contributed' keyword based on bugzilla query, please correct if that's not accurate (by marking patches as obsolete and removing the 'contributed' keyword.