fixed remove template action. Registration was missing in struts.xml.

[mbien]

Hello Dave,
It appears that a past commit intended to remove the templatesRemove action but removed templateRemove too (44b008d#diff-207d6f47f5499d9694104c3640c3b379 ).

So i added it again and the action is working again. Since the confirm dialog seems to be client side now I left the cancel/redirect lines out.

[snoopdave]

How do I reproduce the problem that is fixed by this PR?

When I go to the templates page and click the trash-can to remove a template I get a client-side (non struts) popup that asks "Are you sure you want to remove this template" and the removal is successful.

[mbien]

interesting. If its working something might be wrong with my repository. When i pressed trashcan->ok it didn't remove it and the log complained about undefined action.

i think i found the issue, I should have be using the 6.0.x branch, right? I have master built and running.

master:
https://github.com/apache/roller/blob/master/app/src/main/resources/struts.xml#L562
(remove is missing)

6.0.x:
https://github.com/apache/roller/blob/roller-6.0.x/app/src/main/resources/struts.xml#L562
(remove is there)

i am sorry if i caused confusion. Feel free to ignore this PR.

[mbien]

 [roller]$ git branch --list
 * (HEAD detached at origin/roller-6.0.x)
 [roller]$ git cherry -v remotes/origin/master
+ 3bb06f7ba548d221be8a19f0b4ce027ef5d8edb3 Merge AsciiDoc changes into 6.
+ ca02d5c0c7dc9f3b38f31c78985f22f3cebdcbff Fix suggested by Greg Huber.
+ 4373757c98defb39a48e2b300ea367f903a5a974 Implement template remove.
+ f77c3d762c7ecf0bdebb1ace39f5330d05657029 Preparing for  Roller 6 rc3.
- 32051364c5f8ec6adae214e3ecca7e60176aef8c Fixed: Variables scoping in ajax-user.js (ROL-2157) Pattern is identified and reported at  sonacloud.io as Blocker
- 9dbc9aa1b0d3b0932410bab94396bfc5dbae065a Fixed: Variables scoping in roller and searchhi js files (ROL-2157) Pattern is identified and reported at  sonacloud.io as Blocker
- 45d0837dd8e2df57266ac781dbcb6f8ad61566ea Fixed: Variables scoping in weblog.vm js code (ROL-2157) Pattern is identified and reported at  sonacloud.io as Blocker
- d69b27b8f3dae837c91f0675b013b36cde8cf603 Fixed: Variables should be declared explicitly in Custom JS code (ROL-2157)
- bc9a5880bda35fe1c33a72f3d1c8f781a0b699e8 Fix add user page.
- c78fc67380827e6486f83f2f6b77e79f7bb3e5b7 [ROL-2161] Allow user emails up to 255 characters and add validation to User Edit page.
- 5d8c5ff8312f9a90427e1311943123660a91664d [ROL-2160] work around apparent Struts but by showing weblog handle instead of name in front-page select.
+ 0f1edb2a219171d7600325c6ad1b32ece7a355f7 Prepare for 6.0.1 release.
+ b8df01916df4501694c703a2cd0ac5a40fcf4e84 Preparing for 6.0.1 rc-1
+ 5941c751441378ca2d5ab4a97e35dbe5ba27d4ec Fix incorrect  text key, fix suggested by ghuber
+ 0e3aaa24f7c11ea1965e65bb9d352e018dc6f01d Fix incorrect text key, fix suggested by ghuber.

looks like master has a few commits missing, for example:
4373757 Implement template remove

closing this pull request since its redundant with this commit applied.