Title: PHP Syntax issues
Last modified: August 21, 2016

---

# PHP Syntax issues

 *  [Vinny](https://wordpress.org/support/users/usestrict/)
 * (@usestrict)
 * [12 years, 10 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/)
 * Hi,
 * I was asked by my employer to review your code to decide whether we recommend
   it to our clients and two things caught my attention:
    1. Unquoted array indexes: You have several unquoted array indexes across several
       files. These tend to raise E_NOTICE errors and also cause some PHP overhead 
       as it looks for constants before assuming the literal index.
    2. New lines after closing PHP tags: Please remove the newlines after the closing
       PHP tags. Actually, it’s even better if you remove the closing PHP tag altogether.
 * Otherwise the code looks good.
 * Thanks,
    Vinny
 * [http://wordpress.org/plugins/mailchimp-form/](http://wordpress.org/plugins/mailchimp-form/)

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

 *  [JR1987](https://wordpress.org/support/users/jr1987/)
 * (@jr1987)
 * [12 years, 10 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/#post-4157750)
 * Hi there Vinny
 * Thanks so much for your valuable feedback. I’ll let my co workers at developing
   about your observations. These will help us to improve the plugin’s functionalities.
 * Have an awesome day.
 *  Thread Starter [Vinny](https://wordpress.org/support/users/usestrict/)
 * (@usestrict)
 * [12 years, 10 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/#post-4157819)
 * Thanks for the quick reply.
 * I’d also like to add that the plugin is causing duplicate “Settings Saved” messages
   whenever I save something under Settings. I just found this out today and have
   not looked for the root cause.
 * Cheers,
    Vinny
 *  [JR1987](https://wordpress.org/support/users/jr1987/)
 * (@jr1987)
 * [12 years, 9 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/#post-4157875)
 * Hi there Vinny
 * Can you share a screenshot of this duplicate settings message please? this with
   the objective of sharing it with developing and take a look at what may be causing
   it.
 * Thanks
 *  Thread Starter [Vinny](https://wordpress.org/support/users/usestrict/)
 * (@usestrict)
 * [12 years, 9 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/#post-4157876)
 * There you go:
 * [http://usestrict.net/wp-content/uploads/2013/10/Screen-Shot-2013-10-22-at-6.33.06-PM.png](http://usestrict.net/wp-content/uploads/2013/10/Screen-Shot-2013-10-22-at-6.33.06-PM.png)
 *  [JR1987](https://wordpress.org/support/users/jr1987/)
 * (@jr1987)
 * [12 years, 9 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/#post-4157883)
 * Hi there,
 * We’re looking into this, and will come back to you once we have an answer from
   developing.
 * Very best.

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

The topic ‘PHP Syntax issues’ is closed to new replies.

 * ![](https://s.w.org/plugins/geopattern-icon/mailchimp-form_262627.svg)
 * [MailChimp Form by ContactUs ===Contributors: contactus.com](https://wordpress.org/plugins/mailchimp-form/)
 * [Support Threads](https://wordpress.org/support/plugin/mailchimp-form/)
 * [Active Topics](https://wordpress.org/support/plugin/mailchimp-form/active/)
 * [Unresolved Topics](https://wordpress.org/support/plugin/mailchimp-form/unresolved/)
 * [Reviews](https://wordpress.org/support/plugin/mailchimp-form/reviews/)

 * 5 replies
 * 2 participants
 * Last reply from: [JR1987](https://wordpress.org/support/users/jr1987/)
 * Last activity: [12 years, 9 months ago](https://wordpress.org/support/topic/php-syntax-issues-1/#post-4157883)
 * Status: not resolved