This is on 0.4-alpha8

I just had a user attempt to create a domain name with uppercase characters in it and it didn't work so well :). This was using the site signup form. Checking the code, I noticed that in hosting_site_nodeapi you use a "case submit", but according to the nodeapi docs submit isn't a valid op. Also, even if it was I think it would still miss submissions from the signup form as the site node form isn't being submitted in that case.

This patch replaces the submit case with a presave case.

CommentFileSizeAuthor
hosting_site_lower_title.patch420 byteshadsie

Comments

adrian’s picture

adrian commented
Status: Needs review » Fixed

[master 741e5a3] Fixed the strtolower not being called on site titles. Thanks @hadsie - #830866
1 files changed, 1 insertions(+), 1 deletions(-)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 58255f6 on 6.x-2.x, 7.x-3.x, dev-ssl-ip-allocation-refactor, dev-sni, dev-helmo-3.x by adrian: 
    Fixed the strtolower not being called on site titles. Thanks @hadsie - #...

  • Commit 58255f6 on 6.x-2.x, 7.x-3.x, dev-ssl-ip-allocation-refactor, dev-sni, dev-helmo-3.x by adrian: 
    Fixed the strtolower not being called on site titles. Thanks @hadsie - #...