New project wizard should trim URL

Bug #1170256 reported by Alexander Taler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar Plugin for Eclipse
Fix Released
Low
Piotr Piastucki
Java library for Bazaar
Fix Released
Low
Piotr Piastucki

Bug Description

When copying a URL from e.g. Launchpad, it's easy to get trailing or leading whitespace on the URL. This is almost always an error, so the new project wizard should remove that whitespace rather than producing cryptic error messages.

If trailing whitespace is required on the URL then it can be done using URL encoding, like %20.

To reproduce the problem, try creating a project from this URL, copied from the bzr-java-lib site:

"lp:bzr-java-lib "

Changed in bzr-eclipse:
milestone: none → 1.2
Changed in bzr-eclipse:
status: New → Confirmed
affects: bzr-eclipse → bzr-java-lib
Changed in bzr-java-lib:
milestone: 1.2 → none
milestone: none → 1.2
assignee: nobody → Piotr Piastucki (piastucki)
importance: Undecided → Low
status: Confirmed → In Progress
Changed in bzr-java-lib:
status: In Progress → Fix Committed
Revision history for this message
Alexander Taler (alex-idereal) wrote :

The fix in bzr-java-lib looks correct, however, it should probably be trimmed in the project wizard as well, since the current implementation will create projects and directories with a space on the end.

Changed in bzr-java-lib:
status: Fix Committed → In Progress
status: In Progress → Fix Committed
Changed in bzr-eclipse:
status: New → In Progress
Changed in bzr-eclipse:
milestone: none → 1.2
assignee: nobody → Piotr Piastucki (piastucki)
importance: Undecided → Low
Revision history for this message
Piotr Piastucki (piastucki) wrote :

I hope it is not correctly fixed in new project wizard too.

Changed in bzr-eclipse:
status: In Progress → Fix Committed
Revision history for this message
Piotr Piastucki (piastucki) wrote :

I hope it is *NOW* correctly fixed in new project wizard too. :)

Revision history for this message
Alexander Taler (alex-idereal) wrote :

Yes confirmed it works well now

Changed in bzr-java-lib:
status: Fix Committed → Fix Released
Changed in bzr-eclipse:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.