Forum Replies Created

Viewing 9 replies - 1 through 9 (of 9 total)
  • Thread Starter fredsbend

    (@fredsbend)

    I guess I had the exact same issue 7 years ago.

    Except this time, the plugin needs to be changed in the following way:

    Line 46 in the plug in code can be changed from:

    $out = '<' . sanitize_key( $atts['list_type'] ) . ' class="' . esc_attr( $atts['class'] ) . '">' . $out . '</' . sanitize_key( $atts['list_type'] ) . '>';

    To:

    $out = '<ol class="' . esc_attr( $atts['class'] ) . '">' . $out . '</ol>';

    This hard codes all lists made by this shortcode to <ol>. You can alternatively add the following to your CSS and add a class to the lists, as the linked answer above notes.

    ul.list-pages-shortcode {list-style-type: decimal;}
    Thread Starter fredsbend

    (@fredsbend)

    @cofhugo Sorry, I’m kind of flying by the seat of my pants here already. I’m better than novice with this coding stuff, but still very much an amateur. I only noticed your comment because I came back to find out what I did. The last update undid my changes. I hope you worked out your issue, and maybe it was even a better solution than mine.

    For others reading, xoo-el-login-section.php is found in /wp-content/plugins/easy-login-woocommerce/templates/global/

    I also found it useful to update line 75 in class-xoo-el-form-handler.php found in /wp-content/plugins/easy-login-woocommerce/includes/

    $success_notice = apply_filters( 'xoo_el_login_success_notice', '<i class="fa fa-check-circle" aria-hidden="true"></i> '.__( 'Login successful', 'easy-login-woocommerce' ), $user );
    to 
    $success_notice = apply_filters( 'xoo_el_login_success_notice', '<i class="fa fa-check-circle" aria-hidden="true"></i> '.__( 'Login successful <p><a href="/my-account/">Click here if you are not redirected in a few seconds.</a></p>', 'easy-login-woocommerce' ), $user );
    • This reply was modified 6 years ago by fredsbend.
    Thread Starter fredsbend

    (@fredsbend)

    Okay, I wasn’t sure if I broke something unintentionally. I wasn’t sure what the original behavior was before I started fiddling. I deleted the plugin and reinstalled. Original behavior was that login on Desktop and mobile was successful, but redirect would not happen on either.

    I have accomplished a redirect on both, but there’s some style issues I might work on in the future.

    Here’s what I changed:

    xoo-el-login-section.php Line 60

    <input type="hidden" name="redirect" value="<?php echo $redirect; ?>">
    	changed to
    <input type="hidden" name="redirect" value="/my-account/">

    A few other items I hard code changed as well, but they were simple message type things, piggybacked on other messages already in your code.

    At this point, it’s working well enough for me, so you don’t have to commit any more time to this support request. Thank you for your help.

    Thread Starter fredsbend

    (@fredsbend)

    Okay, I now have the redirect hard coded.

    Line 76 in class-xoo-el-form-handler.php says:

    'notice' => xoo_el_add_notice('success','<i class="fa fa-check-circle" aria-hidden="true"></i> <p><a href="/my-account/">Click here if you are not redirected in a few seconds.</a></p><script>setTimeout(window.location = "/my-account/", 1000);</script>'.__('Login successful')),'redirect' => $redirect);

    It will redirect now, but the styles are goofy.

    Mobile still just spinning. I’ve only tested Android Chrome, btw.

    Thread Starter fredsbend

    (@fredsbend)

    My website

    You can click the login link at the top menu, or on that page itself.

    Thank you,

    Thread Starter fredsbend

    (@fredsbend)

    I’ve edited the plugin php directly with the following code. It was able to update those fields now:

    				// (string, comma separated) group access
    				$group_access = $h->get_data($this,$data,'group_access');
    				if ($group_access) {
    					$post['group_access'] = $group_access;
    				}
                      echo $group_access;
                      echo ' ';
    
    				// (string, comma separated) tag list
    				$tag_list = $h->get_data($this,$data,'tag_list');
    				if ($tag_list) {
    					$post['tag_list'] = $tag_list;
    				}
                      echo $tag_list;
    
    global $wpdb;
    $dbresult = $wpdb->update($wpdb->posts, ['group_access' => $group_access, 'tag_list' => $tag_list], ['ID' => $post_id]);
    if (false === $dbresult) {
        echo 'An error occurred while updating...';
        $errors = $post_id->get_error_messages();
    }

    It does not edit the fields if creating a new entry. I’m working on it. In the meantime, I just import twice, the first to create the rows, the second to make the updates to these specific fields.

    It would be great if the plugin author would add the functionality for the importer to account for user specific fields, instead of just the default WP fields.

    • This reply was modified 9 years, 3 months ago by fredsbend.
    • This reply was modified 9 years, 3 months ago by fredsbend.
    Thread Starter fredsbend

    (@fredsbend)

    I figured it out.

    Yes, the column heading for a page theme template is _wp_page_template. And the name of the templates you should use is the exact name of the php file. I had a stupid error preventing it from working. I accidentally added a space at the end of the heading title.

    Thread Starter fredsbend

    (@fredsbend)

    Alternatively, if you don’t want to change the plugin code, you can just target the generated list with css

    ul.list-pages-shortcode {list-style-type: decimal;}

    For me, I didn’t want more stray css in my already very messy css files.

    Thread Starter fredsbend

    (@fredsbend)

    I currently only have a need for ordered lists, so I’ve changed the code on line 97 from:

    $out = '<' . $atts['list_type'] . ' class="' . $atts['class'] . '">' . $out . '</' . $atts['list_type'] . '>';

    to

    $out = '<ol' . ' class="' . $atts['class'] . '">' . $out . '</ol>';

    This hard codes all lists from this shortcode to be ordered lists regardless of what the list-type attribute is.

    I couldn’t figure out how to fix it. It would be nice to have unordered lists as an option available to me in the future.

Viewing 9 replies - 1 through 9 (of 9 total)