• Xander

    (@xander)


    One of the very last things I am looking into before I go and make my site public is the commenting system. Imagine my surprise when I go to test it out and find that if any of the info isn’t correct, wordpress spits out a die() with a line of text on a blank screen.

    What the hell? The code:

    if ( get_settings(‘require_name_email’) && !$user_ID ) {
    if ( 6 > strlen($comment_author_email) || ” == $comment_author )
    die( __(‘Error: please fill the required fields (name, email).’) );
    elseif ( !is_email($comment_author_email))
    die( __(‘Error: please enter a valid email address.’) );
    }

    Is this for real!? I thought this must be a joke, but then I tried commenting on a few blogs people had posted in the Your WordPress section and, sure enough… am I alone in thinking this is rather shoddy?

    I’m wondering if there are any wholesale replacements of the whole system, since this seems like bunk.

Viewing 3 replies - 1 through 3 (of 3 total)
  • whooami

    (@whooami)

    well its not too stylish, obviously ?? however i think the majority of us think that its trivial to worry about what people see that cannot follow directions ?? (least i dont)

    I took to editing a great deal of the little details on my own site. Yes, I was aware of that, and yes I left it as is. I had bigger fish to fry though I guess.

    Noticing that you put this in the plugins and hacks area though — Im curious, you do realize that a full-scale overhaul isnt neccessary to spruce that up, right?

    Phunky

    (@phunky)

    Yeah, its not really much work to change it.

    Replace die() with break; and add a nice bit to the template ??

    Thread Starter Xander

    (@xander)

    It might not be much work, but it still seems bizarre that such a thing would be left out of something that is supposedly “state of the art”… error handling is a very basic part of user interface design, and in this respect it’s as if whoever was coding this decided not to bother.

    Ah well, I’ll sort it out… just another one of many fixes I have to do before I can go public.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Comment System Replacement’ is closed to new replies.