[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
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