Uploaded image for project: 'Spring Tool Suite'
  1. Spring Tool Suite
  2. STS-566

Template wizard should not allow project names with spaces when creating a bundle project.

    Details

      Description

      Bundles forbid the use of spaces (among other characters) in their IDs. When running the template project wizard, the wizard uses the project's name as the bundle ID, but it does not filter the project name for invalid characters. This can lead to creating a brand new project with compile errors because the bundle ID is invalid. The template project wizard should filter out invalid characters, or convert them into underscores as the bundle project wizard does.

        Activity

        Hide
        cdupuis Christian Dupuis added a comment -

        Can't we strict the the regexp on the template's project name field only allow valid characters?

        Show
        cdupuis Christian Dupuis added a comment - Can't we strict the the regexp on the template's project name field only allow valid characters?
        Hide
        ldossantos Leo Dos Santos (c) [X] added a comment -

        Yes, that's the idea. Either add a regexp on the field to prevent a user from typing invalid characters. Or we could do what the Bundle & PDE project wizards do, which is to replace invalid characters with underscores in the bundle ID.

        Show
        ldossantos Leo Dos Santos (c) [X] added a comment - Yes, that's the idea. Either add a regexp on the field to prevent a user from typing invalid characters. Or we could do what the Bundle & PDE project wizards do, which is to replace invalid characters with underscores in the bundle ID.
        Hide
        thon Terry Denney added a comment -

        I've added a validation to disable Finish or Next if project name contains Space. I'm marking this as fixed..reopen if this is not a good enough solution.

        Show
        thon Terry Denney added a comment - I've added a validation to disable Finish or Next if project name contains Space. I'm marking this as fixed..reopen if this is not a good enough solution.

          People

          • Assignee:
            Unassigned
            Reporter:
            ldossantos Leo Dos Santos (c) [X]
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: