Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Petclinic - Apache OJB in tutorial [SPR-3141] #7827

Closed
spring-projects-issues opened this issue Feb 10, 2007 · 2 comments
Closed

Petclinic - Apache OJB in tutorial [SPR-3141] #7827

spring-projects-issues opened this issue Feb 10, 2007 · 2 comments
Labels
type: bug A general bug
Milestone

Comments

@spring-projects-issues
Copy link
Collaborator

Claus Ibsen opened SPR-3141 and commented

I deployed the Petclinic sample in Tomcat and was reading the Petclinic tutorial that is provided with this sample.

#1
The tutorial mentions Apache OJB that to my recall has been removed from Spring 2.0 onwards and now resides in Spring Modules.
I think the tutorial should be updated and have Apache OJB removed.

#2
There is a note in the tutorial mentioned that iBATIS is not demonstrated in this sample
( NOTE: Spring also provides infrastructure for using other Object-Relational-Mapping frameworks such as JDO and iBATIS SqlMaps but these are not demonstrated in PetClinic. )

I think the note should point to the other sample jpetstore that has iBATIS demonstrated.


Affects: 2.0.2

@spring-projects-issues
Copy link
Collaborator Author

Rick Evans commented

Fixed, thanks again for spotting this.

(Actually, having been forced to actually read the tutorial for the PetClininc, it's a bit shabby. Someone went crazy with the bold tag and the tutorial is darn hard to read. A few UML diagrams would be much better suited and then a lot of that text can just be deleted. I'll have a butchers at revising the sample application in the next few months.)

Cheers
Rick

@spring-projects-issues
Copy link
Collaborator Author

Claus Ibsen commented

Rick your constant effort on improving/fixing the Spring documentation is really appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

1 participant