diff options
| author | Denis M. (Phr33d0m) <god@politeia.in> | 2013-10-13 15:42:12 +0200 |
|---|---|---|
| committer | Denis M. (Phr33d0m) <god@politeia.in> | 2013-11-04 23:59:55 +0100 |
| commit | 8267407a78375652859f9e6b0a3a6fbabdd4a98b (patch) | |
| tree | 423533aa212fb2a12f3aaac05af8567168b8b478 /modules/extra/webcpanel/pages | |
| parent | 6e8d2bf09dbc349df5e2d2094a83a60c6309b76b (diff) | |
Add a couple of usability and aesthetic changes, fix the half-broken chanlist functionality, add a couple of missing ESCAPED_CHANNEL vars
Add missing ESCAPED_CHANNEL
Changed EMAIL_TYPE to FORCE_EMAIL. Now makes more sense
Make it more obvious when the 'email' field is optional
Add a 'Retry' button to go back to the registration page
Fixed broken chanlist functionality. Before even if the user clicked on the channels nothing happened, now it goes straight to the 'set' page (at least something happens)
Small fix, disable the 'Save' button if user has no SET powers
Add another missing ESCAPED_CHANNEL
Add a more clear definition of what hostmasks are allowed in that field, either user@host or nick\!user@host
Fix wrongly placed var
Changed some aesthetics
Diffstat (limited to 'modules/extra/webcpanel/pages')
| -rw-r--r-- | modules/extra/webcpanel/pages/chanserv/drop.cpp | 7 | ||||
| -rw-r--r-- | modules/extra/webcpanel/pages/chanserv/set.cpp | 1 | ||||
| -rw-r--r-- | modules/extra/webcpanel/pages/register.cpp | 4 |
3 files changed, 9 insertions, 3 deletions
diff --git a/modules/extra/webcpanel/pages/chanserv/drop.cpp b/modules/extra/webcpanel/pages/chanserv/drop.cpp index 21e968c18..c6984988c 100644 --- a/modules/extra/webcpanel/pages/chanserv/drop.cpp +++ b/modules/extra/webcpanel/pages/chanserv/drop.cpp @@ -42,7 +42,12 @@ bool WebCPanel::ChanServ::Drop::OnRequest(HTTPProvider *server, const Anope::str } if (message.get_data.count("channel") > 0) - replacements["CHANNEL_DROP"] = message.get_data["channel"]; + { + const Anope::string &chname = message.get_data["channel"]; + + replacements["CHANNEL_DROP"] = chname; + replacements["ESCAPED_CHANNEL"] = HTTPUtils::URLEncode(chname); + } TemplateFileServer page("chanserv/drop.html"); page.Serve(server, page_name, client, message, reply, replacements); diff --git a/modules/extra/webcpanel/pages/chanserv/set.cpp b/modules/extra/webcpanel/pages/chanserv/set.cpp index 35861ad19..dba11557c 100644 --- a/modules/extra/webcpanel/pages/chanserv/set.cpp +++ b/modules/extra/webcpanel/pages/chanserv/set.cpp @@ -110,6 +110,7 @@ bool WebCPanel::ChanServ::Set::OnRequest(HTTPProvider *server, const Anope::stri replacements["SUCCESSOR"] = ci->GetSuccessor()->display; replacements["TIME_REGISTERED"] = Anope::strftime(ci->time_registered, na->nc); replacements["LAST_USED"] = Anope::strftime(ci->last_used, na->nc); + replacements["ESCAPED_CHANNEL"] = HTTPUtils::URLEncode(chname); if (!ci->last_topic.empty()) { diff --git a/modules/extra/webcpanel/pages/register.cpp b/modules/extra/webcpanel/pages/register.cpp index 730db612d..c2c5d5d31 100644 --- a/modules/extra/webcpanel/pages/register.cpp +++ b/modules/extra/webcpanel/pages/register.cpp @@ -13,8 +13,8 @@ bool WebCPanel::Register::OnRequest(HTTPProvider *server, const Anope::string &p replacements["TITLE"] = page_title; - if (!Config->GetModule("nickserv")->Get<bool>("forceemail", "yes")) - replacements["EMAIL_TYPE"] = "hidden"; + if (Config->GetModule("nickserv")->Get<bool>("forceemail", "yes")) + replacements["FORCE_EMAIL"] = "yes"; TemplateFileServer page("register.html"); |