• Resolved loukky

    (@loukky)


    The latest version,when i test request sub i found this issue,Maybe it is a bug?
    Some mail address showed its not valid.
    like someone@students.solano.edu

Viewing 15 replies - 31 through 45 (of 56 total)
  • Thread Starter loukky

    (@loukky)

    Do it, please test again.

    Plugin Author WPKube

    (@wpkube)

    This is what I’m getting:

    But there was no reCAPTCHA functionality in version 200813 so that can’t happen.

    Are you sure version ‘200813` is installed at the moment?

    Can you copy/paste the contents of the file request-management-link.php here?

    Thread Starter loukky

    (@loukky)

    request-management-link.php
    is the latest

    • This reply was modified 4 years, 5 months ago by loukky.
    • This reply was modified 4 years, 5 months ago by loukky.
    Plugin Author WPKube

    (@wpkube)

    Use all the files from 200813. No files should be from the latest version.

    Sorry, I should have been more clear. When I said install version 200813 I meant the whole plugin should be 200813, not a specific file.

    • This reply was modified 4 years, 5 months ago by WPKube.
    • This reply was modified 4 years, 5 months ago by WPKube.
    Thread Starter loukky

    (@loukky)

    DONE,
    All file from 200813 now

    • This reply was modified 4 years, 5 months ago by loukky.
    Plugin Author WPKube

    (@wpkube)

    Great, now we can add changes that happened after that version (the changes that can be related to the issue) one by one to see which one will get the issue to show up.

    First change that was done was on request-management-link.php file. Replace that file with this https://github.com/stcr/subscribe-to-comments-reloaded/blob/271444527e67a99ccf416771b90156915c729860/src/templates/request-management-link.php

    Once you do that try the form. Let me know if the issue shows up.

    • This reply was modified 4 years, 5 months ago by WPKube.
    Thread Starter loukky

    (@loukky)

    Done..
    It showed
    Email address is not valid

    Plugin Author WPKube

    (@wpkube)

    Ok. That means request-management-link.php was being loaded even before 200813 but it simply didn’t show up as an error because in that version we weren’t checking if the email address is connected to an existing subscription.

    That file is not supposed to load.

    On line 869 in wp_subscribe_reloaded.php (it’s a file at the top level, so it’s wp-content/plugins/subscribe-to-comments-reloaded/wp_subscribe_reloaded.php) you’ll see:

    if (empty($include_post_content)) {

    Change that to:

    if ( empty($include_post_content) && empty( $action ) ) {

    Then try again.

    If the error does not show up, update to the latest version of the whole plugin and apply that last change (the one mentioned just above) to the latest version and try the form again.

    • This reply was modified 4 years, 5 months ago by WPKube.
    Thread Starter loukky

    (@loukky)

    Done…
    When i click send button…
    It is no error..
    But blank…
    And no email…
    Maybe you need to test
    link:
    https://loukky.com/comment-subscriptions?srp=1868&srk=c588b142542d2f32de761a79bc143aec&sra=s&srsrc=f#gsc.tab=0

    Plugin Author WPKube

    (@wpkube)

    In subscribe.php on line 30 is:

    ob_start();

    Make that

    ob_start();
    echo 'test 1';

    And on line 233 is:

    $output = ob_get_contents();

    Make that:

    echo 'test2';
    $output = ob_get_contents();

    Try the form submission, does it show both test1 and test2 in there?

    Thread Starter loukky

    (@loukky)

    Before click send button,
    It showed test1 and test2,
    After click send button.
    It showed nothing

    Plugin Author WPKube

    (@wpkube)

    Line 813 of wp_subscribe_reloaded.php is:

    $include_post_content = include WP_PLUGIN_DIR . '/subscribe-to-comments-reloaded/templates/subscribe.php';

    Before that line add:

    echo 'email is ' . $email;
    echo '<br>test3';

    Let me know what shows after submission.

    • This reply was modified 4 years, 5 months ago by WPKube.
    Thread Starter loukky

    (@loukky)

    it showed

    email is someone@students.solano.edu
    test3
    Thread Starter loukky

    (@loukky)

    Create a discod channel to community?

    Plugin Author WPKube

    (@wpkube)

    In subscribe.php on line 30 is :

    ob_start();

    Make that

    return 'test5: ' . $email;
    ob_start();
    • This reply was modified 4 years, 5 months ago by WPKube.
Viewing 15 replies - 31 through 45 (of 56 total)

The topic ‘Email address is not valid’ is closed to new replies.