github: shadowy octopus with the head of a robot, emblazoned with the Dreamwidth swirl (Default)
github ([personal profile] github) wrote in [site community profile] changelog2014-10-16 07:29 am

[dreamwidth/dw-free] 8d7f3c: Convert support/submit.bml+widget to controller+TT

Branch: refs/heads/develop
Home: https://github.com/dreamwidth/dw-free
Commit: 8d7f3c1a793e0b69960031d677948b7e90f8efd0
https://github.com/dreamwidth/dw-free/commit/8d7f3c1a793e0b69960031d677948b7e90f8efd0
Author: Pau Amma <pauamma@dreamwidth.org>
Date: 2014-10-16 (Thu, 16 Oct 2014)

Changed paths:
M bin/upgrading/deadphrases.dat
M bin/upgrading/en.dat
M cgi-bin/DW/Captcha.pm
A cgi-bin/DW/Controller/Support/Submit.pm
R cgi-bin/LJ/Widget/SubmitRequest.pm
R cgi-bin/LJ/Widget/SubmitRequest/Support.pm
R htdocs/support/submit.bml
R htdocs/support/submit.bml.text
A views/support/submit.tt
A views/support/submit.tt.text

Log Message:
-----------
Convert support/submit.bml+widget to controller+TT

- DW::Controller::Support::Submit, views/support/submit.tt, and
views/support/submit.tt.text

Note 3 deviations from the original HTML here:

1- I didn't bother to include an id= in the div because the original id
changes between reloads, and thus isn't usable for styling, and there's
no JS making use of it that I can see.

2- I got rid of the unused, incorrect classname per request.

3- Also per request, I used DW::FormErrors (with some minor deviations
from recommended use to get errors to display since this isn't a
Foundation page) instead of reverse-engineering the HTML the original
BML magic used..

Also, this doesn't implement the support_request_cat_extra_text hook
(which wasn't defined anywhere that I could see), or the invisible
category system half-implemented in LJ::Widget::SubmitRequest->category
and LJ::Widget::SubmitRequest->render_body (as the one existing subclass
doesn't override it).

- Add example of CAPTCHA use in controllers+templates to DW::Captcha. While
I'm at it, get rid of BML dependency.

- Get rid of more strings while I'm at it. These were never used as far as I
can tell.

- Remove widget files no longer needed: LJ::Widget::SubmitRequest::Support
and LJ::Widget::SubmitRequest

Fixes #963


Commit: 565f7f38ea960e81ed9c3aade9b4106ace3a1c57
https://github.com/dreamwidth/dw-free/commit/565f7f38ea960e81ed9c3aade9b4106ace3a1c57
Author: Afuna <afuna@users.noreply.github.com>
Date: 2014-10-16 (Thu, 16 Oct 2014)

Changed paths:
M bin/upgrading/deadphrases.dat
M bin/upgrading/en.dat
M cgi-bin/DW/Captcha.pm
A cgi-bin/DW/Controller/Support/Submit.pm
R cgi-bin/LJ/Widget/SubmitRequest.pm
R cgi-bin/LJ/Widget/SubmitRequest/Support.pm
R htdocs/support/submit.bml
R htdocs/support/submit.bml.text
A views/support/submit.tt
A views/support/submit.tt.text

Log Message:
-----------
Merge pull request #979 from pauamma/bug-963-support-submit.bml-to-TT

Bug 963 support submit.bml to tt


Compare: https://github.com/dreamwidth/dw-free/compare/d5e462e3aa1a...565f7f38ea96