fix: prevent errors/login screens trying to access donations via email access

[Benunc]

Bug Report

User Story

As a donor, I want to access donation history via email access without creating a user so that I can see my receipt on the web.

Current Behavior

Currently, I get the email about my donation, and I follow the link to view it in my browser (in a browser without the session active). That does not work, and I am greeted with an email access form. I fill out the form, click the link in my email, and then I'm given an error "you must be logged in to view this donation" but I don't have an account at all, so that won't ever work.

Expected Behavior

I expect the link from my email to work the first time.

Bug Type

• This bug describes functionality that once worked as expected in some version.
• This bug describes functionality that never worked as expected.
• I am not sure whether this functionality ever worked as expected.

Steps to Reproduce

1. Create a donation manually (front end or back end) without creating a WP user, with email access enabled.
2. send the email receipt
3. Click the link to go back to the web version of the receipt, in a browser where you didn't complete the donation (session no longer active)
4. The donation history page has an email access form.
5. Fill out that form.
6. Check your email and click that link
7. Login page.

Possible Solution

Related

https://secure.helpscout.net/conversation/753456694/32494/
https://secure.helpscout.net/conversation/731484225/31090/

Acceptance Criteria

• the initial click back to the site from an email gets you access to the donation history
• when email access is enabled, it never asks for logins.

Environment

WordPress System Info

### WordPress Environment ###

Home URL: http://localgive.local
Site URL: http://localgive.local
WP Version: 5.0.3
WP Multisite: –
WP Memory Limit: 256 MB
WP Debug Mode: ✔
WP Cron: ✔
Language: en_US
Permalink Structure: /%year%/%monthnum%/%day%/%postname%/
Show on Front: posts
Table Prefix Length: wp_
Table Prefix Length: 3
Table Prefix Status: Acceptable
Admin AJAX: Accessible
Registered Post Statuses: publish, future, draft, pending, private, trash, auto-draft, inherit, request-pending, request-confirmed, request-failed, request-completed, refunded, failed, revoked, cancelled, abandoned, processing, preapproval, give_subscription

Server Environment

Hosting Provider: DBH: localhost, SRV: localgive.local
TLS Connection: Connection uses TLS 1.2
TLS Connection: Probably Okay
Server Info: nginx/1.10.1
PHP Version: 7.2.0
PHP Post Max Size: 1,000 MB
PHP Time Limit: 900
PHP Max Input Vars: 3000
PHP Max Upload Size: 1,000 MB
cURL Version: ❌ 7.38.0, OpenSSL/1.0.1t - We recommend a minimum cURL version of 7.40.
SUHOSIN Installed: –
MySQL Version: ❌ 5.5.60 - We recommend a minimum MySQL version of 5.6. See: WordPress Requirements
Default Timezone is UTC: ✔
fsockopen/cURL: ✔
SoapClient: ✔
DOMDocument: ✔
gzip: ✔
GD Graphics Library: ✔
Multibyte String: ✔
Remote Post: ✔
Remote Get: ✔

Give Configuration

Give Version: 2.4.0
Give Cache: Enabled
Database Updates: All DB Updates Completed.
Give Cache: Enabled
Give Cache: ✔New Donation✔Donation Receipt✔New Offline Donation✔Offline Donation Instructions✔New User Registration✔User Registration Information✔Donor Note✔Email access❌Renewal Receipt Email❌Renewal Receipt Email❌Subscription Cancelled Email❌Subscription Cancelled Email❌Subscription Completed Email❌Subscription Reminder Email✔Subscriptions Email Access
Upgraded From: –
Test Mode: Enabled
Currency Code: USD
Currency Position: Before
Decimal Separator: .
Thousands Separator: ,
Success Page: http://localgive.local/donation-confirmation/
Failure Page: http://localgive.local/donation-failed/
Donation History Page: http://localgive.local/donation-history/
Give Forms Slug: /donations/
Enabled Payment Gateways: Test Donation, Offline Donation, Stripe - Credit Card
Default Payment Gateway: Stripe - Credit Card
PayPal IPN Verification: Enabled
PayPal IPN Notifications: N/A
Donor Email Access: Enabled

Active Give Add-ons

Give - Authorize.net Gateway: ❌ Unlicensed – by GiveWP – 1.4.5
Give - Manual Donations: ❌ Unlicensed – by WordImpress – 1.4.1
Give - PDF Receipts: ❌ Unlicensed – by GiveWP – 2.3.2
Give - Recurring Donations: ❌ Unlicensed – by GiveWP – 1.8.3
Give - Stripe Gateway: ❌ Unlicensed – by GiveWP – 2.1.3
Give - Tributes: ❌ Unlicensed – by GiveWP – 1.5.2

Other Active Plugins

Ben's Helper Functions: by BenUNC –

Inactive Plugins

Better Search Replace: by Delicious Brains – 1.3.2
Debug Bar: by wordpressdotorg – 1.0
Debug Bar Post Meta: by whyisjake – 0.5.5
Divi Builder: by Elegant Themes – 2.3.1
Duplicate Post: by Enrico Battocchi – 3.2.2
Elegant Themes Support: by Elegant Themes – 1.3
Fusion Builder: by ThemeFusion – 1.0.6
Fusion Core: by ThemeFusion – 3.0.6
Give - 2Checkout Gateway: by WordImpress – 1.0.2
Give - Annual Receipts: by GiveWP – 1.0.0
Give - Braintree Gateway: by WordImpress – 1.2.2
Give - CCAvenue Gateway: by GiveWP – 1.0.2
Give - Constant Contact: by WordImpress – 1.2.1
Give - ConvertKit: by WordImpress – 1.0.2
Give - Currency Switcher: by GiveWP – 1.3.0
Give - Dwolla Gateway: by WordImpress – 1.1.2
Give - Email Reports: by WordImpress – 1.1.2
Give - Fee Recovery: by WordImpress – 1.7.0
Give - Form Countdown: by WordImpress – 1.0.1
Give - Form Field Manager: by GiveWP – 1.4.0
Give - Google Analytics Donation Tracking: by GiveWP – 1.2.2
Give - Hook Helper: by Ravinder Kumar – 1.0
Give - iATS Payment Solutions: by WordImpress – 1.0.2
Give - MailChimp: by GiveWP – 1.4.3
Give - Old Meta Switcher: by WordImpress – 1.0
Give - PayFast Payment Gateway: by WordImpress – 1.0.2
Give - Paymill Gateway: by WordImpress – 1.1
Give - PayPal Pro Gateway: by WordImpress – 1.1.6
Give - Paytm Gateway: by WordImpress – 1.0
Give - PayUmoney: by WordImpress – 1.0.1
Give - Per Form Gateways: by WordImpress – 1.0.1
Give - Razorpay: by WordImpress – 1.1.3
Give - Recurring Helper: by WordImpress – 1.0
Give - Update Goal Progress: by Matt Cromwell – 1.0
Give - WePay Gateway: by WordImpress – 1.3.1
Give - WP All Import Addon: by Ben Meredith – 1.0
Give - Zapier: by WordImpress – 1.2.1
Give Stripe Cleanup.: by BenUNC –
Hello Dolly: by Matt Mullenweg – 1.6
Transients Manager: by Pippin Williamson – 1.7.5
User Switching: by John Blackbourn & contributors – 1.4.0

Active MU Plugins

Bypass Login: by Stephen Carnam – 1.1.0
hide-give-license-warning.php: by –
Local by Flywheel Relative URL (for Live Links): by Flywheel – 1.0

Theme

Name: Twenty Seventeen
Version: 2.0
Author URL: https://wordpress.org/
Child Theme: No – If you're modifying Give on a parent theme you didn't build personally, then we recommend using a child theme. See: How to Create a Child Theme