ejm

re:

Remember that comment author link output occurs during a GET request, so there will be nothing in the global $_POST.

I’m guessing that this explains (sort of) why a webname value that is removed in WP_admin, after being entered from outset, remains as ' ', rather than disappearing.

Because I have been unable to figure out how to remove the parentheses from those newly empty value fields, I’ve compromised by changing the parentheses to a single tilde: $author .= esc_html(" ~ $webname");. This way, it doesn’t look ridiculous to see “Commenter’sName ~” instead of “Commenter’sName ()”

I’m completely unclear where to put

if ($webname = '') || ($webname = ' ') {
  !isset( $webname ) 
}

(I don’t even know if !isset is an actual thing….)

OR

if ($webname = ' ') {
  $_POST['webname'] = ''; 
}

Also, if $webname = '', is it not possible to display the emptiness with emptiness, rather than a tilde (or as before, inside parentheses)?

I realize that you may well have answered this already; please excuse my obtuseness.
Babysteps! Babysteps! Thank you again for all your help.

• This reply was modified 6 years, 7 months ago by ejm.

Please excuse me for replying to myself….

It’s a miracle! I think I got it… in the function, attach_websitename_to_author( $author ), I have to use if( isset( $_POST['webname'] ) ) instead of if ( $webname ). This is so exciting!

Thank you once again for all your help. It is most appreciated.

function comment_edit_function( $comment_id )
{
    if( isset( $_POST['webname'] ) )
      update_comment_meta( $comment_id, 'webname', esc_attr( $_POST['webname'] ) );
	else 
	$_POST['webname'] = ' ';
}

add_filter( 'get_comment_author_link', 'attach_websitename_to_author' );

function attach_websitename_to_author( $author ) {
    $webname = get_comment_meta( get_comment_ID(), 'webname', true );
	if( isset( $_POST['webname'] ) )
//    if ( $webname )
        $author .= " ($webname)";
	elseif ($_POST['webname'] = ' ')
		$author .= " $webname";
//		$author .= " ";
		return $author;
 }

That took a lot less time than expected! Good idea to add define( 'WP_DEBUG', true ); in my test WP’s config file. That was most helpful. The first thing that came up was a very easy fix on another part of the comment customizaton, once I realized the variable had to be defined inside the curly bracket of the function…

Notice: Undefined variable: commenter in ../wp-content/themes/my-child-theme/functions.php on line 140

function modify_comment_form_fields($fields){
	$commenter = 0;
    $fields['url'] = '<p class="comment-form-url"><label for="url">' . __( 'Path on Web', 'domainreference' ) . '</label>' .
    '<input id="url" name="url" type="text" value="' . esc_attr( $commenter['comment_author_url'] ) .

I was also missing opening curly bracket after if ( isset ( $_POST[‘webname’] )

/*..........
Update comment meta data from comment editing screen 
 ....*/ 
function webname_comment_edit_meta_fields( $comment_id ) {
    if ( isset( $_POST['webname'] ) && '' != $_POST['webname']) {
        update_comment_meta( $comment_id, 'webname', esc_attr( $_POST['webname'] ) );
    }

}

But the main problem and the reason for the inability to edit was because I had the add-action placed erroneously inside final curly bracket of the webname_comment_edit_meta_fields( $comment_id ) function!

add_action( 'edit_comment', 'webname_comment_edit_meta_fields' );

Once again, many many thanks for your patient and very helpful answers, bcworkz. It’s thrilling to be able to mark this as resolved.

Here is the full coding I ended up with (that probably still needs cleaning up, but at least it’s working!):

/*...............
add a field to the comment form
...............*/
function add_comment_fields($newfield) {
    $newfield['webname'] = '<p class="comment-form-author" title="this field is optional"><label for="webname">' . __( 'Your Site\'s Name' ) . '</label>' .
        '<input id="webname" name="webname" type="text" size="30" /></p>';
    return $newfield;
}
add_filter('comment_form_default_fields','add_comment_fields');

function add_comment_meta_values($comment_id) {
   if ( ( isset( $_POST['webname'] ) ) && ( $_POST['webname'] != '') ) {
	$webname = wp_filter_nohtml_kses($_POST['webname']);
        add_comment_meta($comment_id, 'webname', $webname);
    } 
	else    
		$webname = '&nbsp;';
}
add_action ('comment_post', 'add_comment_meta_values', 1);

function comment_edit_function( $comment_id )
{
    if( isset( $_POST['webname'] ) )
      update_comment_meta( $comment_id, 'webname', esc_attr( $_POST['webname'] ) );
    else 
		$_POST['webname'] = '&nbsp;';
}

add_filter( 'get_comment_author_link', 'attach_websitename_to_author' );

function attach_websitename_to_author( $author ) {
    $webname = get_comment_meta( get_comment_ID(), 'webname', true );
    if ( $webname )
        $author .= " ($webname)";
	elseif ($webname = '')
		$author .= " $webname";
   return $author;
 }

// Register meta box on comment edit screen
function webname_comment_edit_add_meta_box() {
    add_meta_box( 'title', __( 'website name', 'text-domain' ), 'webname_comment_meta_box', 'comment', 'normal', 'high' );
}
add_action( 'add_meta_boxes_comment', 'webname_comment_edit_add_meta_box' );

// Callback function for displaying the comment meta box.
function webname_comment_meta_box( $comment ) {
    $webname = get_comment_meta( $comment->comment_ID, 'webname', true );
    wp_nonce_field( 'webname_comment_fields_update', 'webname_comment_fields_update', false );
    ?>
    <p>
        <label for="webname"><?php _e( 'Commenter\'s Site Name', 'text-domain' ); ?></label>
        <input type="text" name="webname" value="<?php echo esc_attr( $webname ); ?>" class="widefat" />
    </p><?php
}

/*..........
Update comment meta data from comment editing screen 
 ....*/ 
function webname_comment_edit_meta_fields( $comment_id ) {
    if ( isset( $_POST['webname'] ) && '' != $_POST['webname']) {
        update_comment_meta( $comment_id, 'webname', esc_attr( $_POST['webname'] ) );
    }

}
add_action( 'edit_comment', 'webname_comment_edit_meta_fields' );

/*.......
 from wpengineer.com/2214/adding-input-fields-to-the-comment-form/
 ....*/ 
function save_comment_meta_data( $comment_id ) {
    add_comment_meta( $comment_id, 'webname', $_POST[ 'webname' ] );
}
add_action( 'comment_post', 'save_comment_meta_data' );

// many thanks to bcworkz for help in fixing adding custom field to comments wordpress.org/support/topic/adding-and-editing-non-required-field-in-comments/

Thank you!

Post the code over at pastebin.com. If you sign up there (for free, but not required), you’d be able to go back and edit your code later on in case you want to show me an updated version.

I have done as you suggested. I suspect that there may be duplicate pieces of code….

Here is what I have in my child functions file: pastebin[dot]com/FJtVCvt2

No matter what I do, I cannot seem to get the field to be editable and if the field is left empty, it simply does not appear at all.

I tried it in a test WP as well, with the same results….

Than you again, for your response.

re:

I’m following your explanation except for how the parentheses appear when the field is not filled in.

At this point, the parentheses do not appear if the field is not filled in. But if I put in an instruction to save an empty string if the value is ”, then wouldn’t it appear in the comment meta box as ()?

function attach_websitename_to_author( $author ) {
    $webname = get_comment_meta( get_comment_ID(), 'webname', true );
    if ( $webname )
        $author .= " ($webname)";
   return $author;
 }

Not knowing how to do it (and having tried a number of incorrect bits of code) I’m guessing that I have to say something like

if ($_POST['webname'] = '')
   echo ' ';

This didn’t throw any errors. But it also doesn’t seem to do anything….

re:

[Y]ou don’t need conditional logic on output, only upon form submit. For output, just echo out whatever is in comment meta, whether it be the saved value or an empty string.

Once again, I understand the logic here, but am not certain of which code sections are used for the form submit. Is it this part??

function add_comment_meta_values($comment_id) {
   if ( ( isset( $_POST['webname'] ) ) && ( $_POST['webname'] != '') ) {
        $webname = wp_filter_nohtml_kses($_POST['webname']);
        add_comment_meta($comment_id, 'webname', $webname);
    } 
}
add_action ('comment_post', 'add_comment_meta_values', 1);

Thank you for your reply, bcworkz.

That makes sense that if the field is empty, no data is sent. And intellectually, I understand what you mean by “Change the logic to always save something as long as the comment is created, regardless of the field’s isset() result. You use isset() to decide if you save an empty string or the passed value, not to decide if anything should be saved at all.”

Alas, I do not know how to implement this.

I thought it might be with the function update_option() but as usual, the codex is written for people who probably already know the answer to the question.

Not to mention that apparently, update_option() “has been deprecated […] in favor of update() method instead” (https://developer.wordpress.org/reference/classes/wp_customize_setting/_update_option/).

Is https://developer.wordpress.org/reference/classes/wpdb/update/ the page that might contain the information I need?

Because I couldn’t wrap my head around the wp developer update page I took a shot and tried removing the “if isset” with the following:

function update_comment_meta_values($comment_id) {
if ( $_POST['webname'] != '') {
        $webname = wp_filter_nohtml_kses($_POST['webname']);
        add_comment_meta($comment_id, 'webname', $webname);
    } 
}
add_action ('comment_post', 'update_comment_meta_values', 1);

It will probably come as no surprise to you that this did not work.

Staring more at the coding at the bottom of the page on wpdb::update, I’m guessing that this contains clues to what I am trying to achieve. Is there another place I can refer to (something a little more along the lines of “wp for dummies”) to glean the answer?

public function update( $table, $data, $where, $format = null, $where_format = null ) {
    if ( ! is_array( $data ) || ! is_array( $where ) ) {
        return false;
    }
 
    $data = $this->process_fields( $table, $data, $format );
    if ( false === $data ) {
        return false;
    }
    $where = $this->process_fields( $table, $where, $where_format );
    if ( false === $where ) {
        return false;
    }
 
    $fields = $conditions = $values = [... etc. etc. ...]

This is great. Thank you for dealing with this so quickly!