Bug 220316 - Can't add GET parameters to URL field of PHP Debug
Summary: Can't add GET parameters to URL field of PHP Debug
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P1 major (vote)
Target Milestone: ---   Edit
Assignee: Michael Spector CLA
QA Contact:
URL:
Whiteboard:
Keywords: ui, usability
Depends on:
Blocks:
 
Reported: 2008-02-25 23:07 EST by Scott Maines CLA
Modified: 2020-05-14 11:08 EDT (History)
2 users (show)

See Also:

Attachments
Screenshots (2 in zip) before and after dialog save (134.10 KB, application/zip)
2008-02-25 23:07 EST, Scott Maines CLA 		no flags 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 Scott Maines CLA 2008-02-25 23:07:25 EST 
Created attachment 90714 [details]
Screenshots  (2 in zip) before and after dialog save

Apparently not possible to add GET parameters to URL field of a PHP Web Page debug configuration, they are stripped off on close. Screen shots enclosed.

Incidentally, why is version 1.5.1 an option in this bug submission form? Page http://download.eclipse.org/tools/pdt/downloads/ indicates 1.0.2 is latest version.
Comment 1 Scott Maines CLA 2008-02-26 13:50:13 EST 
On reflection, IMHO, I think that being unable to use GET params is of severity major.
Comment 2 Scott Maines CLA 2008-02-27 05:25:16 EST 
added keywords
Comment 3 Scott Maines CLA 2008-03-22 19:37:23 EDT 
I rigged it for debugging and had a quick look.  My first impressions are,

* that the ServerLaunchConfigurationTab may pull info from the UI at the wrong time,
* that the manner in which the UI parts out the information (ServerLaunchConfigurationTab.updateURLComponents) does not respect that the "BaseURL" may have a filepath, i.e. that BaseURL UI field should not truncate at the server domain, doing which may be propagating the error.

-SM
Comment 4 Michael Spector CLA 2008-04-21 04:42:05 EDT 
Now it's possible to add query part.
Comment 5 Assaf Almaz CLA 2008-04-29 11:46:28 EDT 
Verified.