search

Letractively / jcatapult

Automatically exported from code.google.com/p/jcatapult
0 stars 0 forks source link

Scaffolder recommends package names with backslashes #44

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Run the scaffolder in WindowsXP

What is the expected output? What do you see instead?
The scaffolder recommends some rather strange package names.

Here's a transcript of my scaffolder session:

C:\projects\buttface>%JCATAPULT_HOME%\scaffolder\bin\scaffold.bat crud
Running ant build
Using Savant to determine project classpath
Using Scaffolder home
[C:\server\jcatapult\jcatapult-1.0-RC1\scaffolder\bin\\..]

Enter the domain class to CRUD scaffold []:
com.texturemedia.buttface.domain.EvolutionAccount
Scaffolding domain object [com.texturemedia.buttface.domain.EvolutionAccount]
Enter the URL prefix of the CRUD [/admin/evolutionAccounts]:
Scaffolding URL prefix [/admin/evolutionAccounts]
Enter the action package
[com\texturemedia\buttface\action.admin.evolutionAccounts]:
Scaffolding action package
[com\texturemedia\buttface\action.admin.evolutionAccounts]
Enter the service package [com\texturemedia\buttface\service]:
Scaffolding service package [com\texturemedia\buttface\service]
Please review the scaffolding configuration

    Type: crud
    Domain class: com.texturemedia.buttface.domain.EvolutionAccount
    URL: /admin/evolutionAccounts
    Action package: com\texturemedia\buttface\action.admin.evolutionAccounts
    Service package: com\texturemedia\buttface\service

    Is this is correct, type yes, if not type no [yes]:

The scaffolder recommends a Action package and a Service package with
backslashes.

What version of the product are you using? On what operating system?
1.0-RC1, WinXPSP2

Please provide any additional information below.
Attached is my suggested patch. I think the call to String.replace(String,
String) didn't effectively kill all of the backslashes. I replaced both
method calls with the String.replace(char, char).

Original issue reported on code.google.com by rtgam...@gmail.com on 20 May 2008 at 5:54

Attachments:

GoogleCodeExporter commented 8 years ago 
This has been fixed.

Original comment by leafkn...@gmail.com on 21 May 2008 at 8:36