Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Checking if Var Exists Before Evaluating #3

Merged
merged 1 commit into from
Aug 19, 2014

Conversation

BFTrick
Copy link
Contributor

@BFTrick BFTrick commented Mar 15, 2014

I get these notices when I try to send an email:

Notice: Undefined index: track-clicks in /Users/patrick/Documents/web/projects/ninja-forms/wp-content/plugins/mailgun/includes/wp-mail.php on line 142

Notice: Undefined index: track-opens in /Users/patrick/Documents/web/projects/ninja-forms/wp-content/plugins/mailgun/includes/wp-mail.php on line 143

I checked to make sure the variable exists before evaluating them and that made them go away. :)

@travelton
Copy link
Contributor

Thanks for the fixing this. We'll pull it and update the plugin soon!

@BFTrick
Copy link
Contributor Author

BFTrick commented Apr 25, 2014

Any update on this? :)

@rclations
Copy link

hey @travelton can we get this pulled and sync'd to wordpress.org ?

Is there anything else left needed on this?

travelton added a commit that referenced this pull request Aug 19, 2014
Checking if Var Exists Before Evaluating
@travelton travelton merged commit e491019 into mailgun:master Aug 19, 2014
@travelton
Copy link
Contributor

Sorry everyone! This has been published to the Wordpress Plugin Directory as version 1.2.

@BFTrick BFTrick deleted the undefined-index branch August 19, 2014 17:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants