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

LPS-94878 (Thin Spring) LPS-95285 (Support PortletMVC4Spring) #279

Conversation

ngriffin7a
Copy link

LPS-94878 Support PortletMVC4Spring in the Spring MVC project template
LPS-95285 As a developer I would like spring mvc portlets to reuse the spring stack provided by Liferay portal

rotty3000 and others added 4 commits June 7, 2019 16:49
LPS-95285 As a developer I would like spring mvc portlets to reuse the spring stack provided by Liferay portal
@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

@ngriffin7a
Copy link
Author

@gamerson This PR builds upon commits by you and @rotty3000.

I did what you asked -- Rather than having multiple templates, I made an uber project-templates-spring-mvc-portlet that supports the following permutations using Velocity conditionals:

framework frameworkDependencies viewType liferayVersion
springportletmvc embedded (thick) jsp 7.0
springportletmvc embedded (thick) thymeleaf 7.0
springportletmvc embedded (thick) jsp 7.1
springportletmvc embedded (thick) thymeleaf 7.1
springportletmvc provided (thin) jsp 7.1
springportletmvc provided (thin) thymeleaf 7.1
springportletmvc embedded (thick) jsp 7.2
springportletmvc embedded (thick) thymeleaf 7.2
springportletmvc provided (thin) jsp 7.2
springportletmvc provided (thin) thymeleaf 7.2
springportletmvc embedded (thick) jsp 7.3
springportletmvc embedded (thick) thymeleaf 7.3
portletmvc4spring embedded (thick) jsp 7.1
portletmvc4spring embedded (thick) thymeleaf 7.1
portletmvc4spring embedded (thick) jsp 7.2
portletmvc4spring embedded (thick) thymeleaf 7.2
portletmvc4spring embedded (thick) jsp 7.3
portletmvc4spring embedded (thick) thymeleaf 7.3
portletmvc4spring embedded (thin) jsp 7.3
portletmvc4spring embedded (thin) thymeleaf 7.3

Note that the "thin" configurations won't work just yet because Ray is still working on getting thin working in the core. Regardless, they're all setup for thin once Ray's work is done.

Also, it's won't be possible to test the 7.3 permutations until the 7.3 DTDs are available.

Finally, if there are failures in the ProjectTemplateFilesTest.java I'd appreciate it if you would fix them for me.

/cc @david-truong

@gamerson
Copy link
Owner

Hey @ngriffin7a

Thanks for the update. I'll work on this this week.

@gamerson
Copy link
Owner

Just started reviewing :)

:octocat: Sent from GH.

@gamerson
Copy link
Owner

Review stage one has passed and now moved over to Lawrence for another round: lawrence-lee#120

Thanks again Neil!

@gamerson gamerson closed this Jun 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants