Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

CRM-18515 - Allow Payment Processor with same name when using multi-domain #8316

Closed
wants to merge 44 commits into from

Conversation

darrick
Copy link
Contributor

@darrick darrick commented May 7, 2016

@civicrm-builder
Copy link

Can one of the admins verify this patch?

@monishdeb
Copy link
Member

monishdeb commented May 9, 2016

@darrick can you please squash multiple commits in this PR into one commit. Here are the steps:

  1. git rebase --interactive
  2. Edit command names accordingly
    pick 1fc6c95 My pull request
    squash 6b2481b Hack hack - will be discarded
    squash dd1475d Also discarded
  3. git push -f origin/CRM-18515

for reference http://stackoverflow.com/questions/9790448/how-to-update-a-pull-request

@derrick
Copy link

derrick commented May 9, 2016

Looks like you meant @darrick ...

@monishdeb
Copy link
Member

oops sorry :)

darrick and others added 18 commits May 9, 2016 14:51
Comment suggests 'c_' to identify a non-CRM connection + timestamp to the hour + connection_id but code used 'con_'

I think the shorter version in comment is better as there is more real information space
This should resolve warnings on systems where MyISAM was default engine.
There were problems in this line:
 * The missing comma (required by Drupal style)
 * The use of literal CtrlA causes phpcs to produce invalid output
mollux and others added 19 commits May 9, 2016 14:55
…ing' un-ticked, but still persists

----------------------------------------
* CRM-18154: Profile custom field option: "Include in multi-record listing" un-ticked, but still persists
  https://issues.civicrm.org/jira/browse/CRM-18154
----------------------------------------
* CRM-18154: Profile custom field option: "Include in multi-record listing" un-ticked, but still persists
  https://issues.civicrm.org/jira/browse/CRM-18154
…uage object and language_list are different in Backdrop
In CRM_Mailing_Event_BAO_Delivered::updateEmailResetDate(), call for
CRM_Core_DAO::executeQuery($query), so the temporary table can be filled.
@darrick
Copy link
Contributor Author

darrick commented May 9, 2016

@monishdeb believe I just made things worse. I tried git rebase --interactive after git checkout orgin/CRM-18515 but only choice was NOOP. I then tried git rebase -i origin/master but it didn't list all the commits. So I used git rebase -i HEAD~7 which gave me all the commits and dropped all the commits except for the main one. Then on git push it said I wasn't up to date. So I chose git pull and then git push.

@monishdeb
Copy link
Member

monishdeb commented May 10, 2016

@darrick .. seems like rebase need to be done against upstream/master not origin/master since later is outdated branch that's why you got unwanted commits piled up on top of your earlier commits. (As i said above on the first step that 'git rebase --interactive' actually does rebasing against upstream/master if you don't mention it) For now its better to close the current pr and create a new branch in your local and make the changes manually as the patch is just a one file change.

@darrick darrick closed this May 10, 2016
@darrick darrick deleted the CRM-18515 branch May 10, 2016 18:30
@darrick
Copy link
Contributor Author

darrick commented May 10, 2016

Cleaned up PR. #8344

@civicrm-builder
Copy link

Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.