Answers.Body: <p>Yes you should fork the repo and create a pull request against the Master branch which i assume is the branch with the issue. Please see this documentation on contributing to CiviCRM <a href="https://wiki.civicrm.org/confluence/display/CRMDOC/Contributing+to+CiviCRM+using+GitHub" rel="nofollow noreferrer">https://wiki.civicrm.org/confluence/display/CRMDOC/Contributing+to+CiviCRM+using+GitHub</a> </p>
  
Answers.ContentLicense: CC BY-SA 3.0
Answers.CreationDate: 2017-04-29T06:49:33.037Z
Answers.LastActivityDate: 2017-04-29T06:49:33.037Z
Answers.UserDisplayName: Seamus Lee
Body: <p>I think there is a typo on line 692 of
  wp-content/plugins/civicrm/civicrm/packages/Net/SMTP.php</p>
  
  <p><code>mMethod</code> should be <code>method</code> in the last line below, yes?</p>
  
  <pre><code>if (is_array($callback)) {
              if (!is_object($callback[0]) || !is_string($callback[1]))
                  return PEAR::raiseError('Bad mMethod callback array');
          }
  </code></pre>
  
  <p>Should I fork the <code>civicrm-packages</code> repository, make the change there and submit a pull request?</p>
  
  <p>Or is CiviCRM getting this code from elsewhere, and it would be more useful to make the change in the original source repo?</p>
  
Tags: <wordpress><smtp>
Title: typo in packages/Net/SMTP.php
ThreadText: # typo in packages/Net/SMTP.php
  
  (Asked by: username_0 on 2017-04-29)
  
  I think there is a typo on line 692 of
  wp-content/plugins/civicrm/civicrm/packages/Net/SMTP.php
  mMethod should be method in the last line below, yes?
  if (is_array($callback)) {
              if (!is_object($callback[0]) || !is_string($callback[1]))
                  return PEAR::raiseError('Bad mMethod callback array');
          }
  
  Should I fork the civicrm-packages repository, make the change there and submit a pull request?
  Or is CiviCRM getting this code from elsewhere, and it would be more useful to make the change in the original source repo?
  
  **Tags:** wordpress, smtp
  
  
  ---
  
  ## 1 answers
  
  ### Answer by username_1 on 2017-04-29. Score: 2
  
  Yes you should fork the repo and create a pull request against the Master branch which i assume is the branch with the issue. Please see this documentation on contributing to CiviCRM <https://wiki.civicrm.org/confluence/display/CRMDOC/Contributing+to+CiviCRM+using+GitHub>
  
  
  ---
  
  