fix formatting and add comments

Update version to 4.3.3

Merge pull request #625 from kurund/upgrade-fixes

fixed upgrade issue

fixed upgrade issue

Merge pull request #624 from colemanw/testFix

Fix is_primary handling

Fix is_primary handling

Merge pull request #620 from pratik-joshi/webtest-fixes

this webtest fix is WRT https://github.com/agh1/civicrm-core/commit/4dfc...

this webtest fix is WRT https://github.com/agh1/civicrm-core/commit/4dfcb7f8 this commit

Merge pull request #615 from pratik-joshi/CRM-12378

CRM-12378 a new webtest helper function introduction which can help dete...

Merge pull request #607 from totten/colbyw-rest

Add test case for REST interface

Merge pull request #605 from kurund/CRM-12452

refixed CRM-12452, normal new contact widget was broken

refixed CRM-12452, normal new contact widget was broken

Merge pull request #585 from agh1/fintyperel-relfirst

Put relationship ahead of account name in assigned financial accounts page (CRM-12474)

Merge pull request #598 from dlobo/CRM-12488

CRM-12488

Merge pull request #601 from dlobo/CRM-12475

CRM-12475

Merge pull request #586 from yashodha/CRM-12401-reminder

CRM-12401

CRM-12475

----------------------------------------
* CRM-12475: Advanced search for CiviCRM ID fails
  http://issues.civicrm.org/jira/browse/CRM-12475

RestTest - Add test case for ping

CRM-12488

----------------------------------------
* CRM-12488: CiviMail delivers only 10 emails on the first round
  http://issues.civicrm.org/jira/browse/CRM-12488

CRM-12488

----------------------------------------
* CRM-12488: CiviMail delivers only 10 emails on the first round
  http://issues.civicrm.org/jira/browse/CRM-12488

RestTest - Add test cases for omitted api_key. Tweak comments.

RestTest - Reformat and split testNotCMSUser in two.

The error expected for testNotCMSUser is currently produced for
"?q=civicrm/$entity/$action" but not "?entity=$x&action=$x", so we should be
testing both situations.

RestTest - Test q=contact/$entity/$action syntax. Test calls with bad entities.

CiviSeleniumSettings - Add adminApiKey

RestTest - Use adminApiKey. Comment out test-cases for deprecated/dead functionality. Simplify/patch remaining test-cases.

RestTest - Move code from setUp() to _setUpAdminSessionIdAndApiKey(). Keep current behavior, but make it easier to change on per-test basis.

Merge pull request #587 from dlobo/FormattingFixes

formatting fixes

formatting fixes

CRM-12401

RestTest - Formatting

RestTest - Simplify test for property "siteKey" settings

WebTest_Utils_RestTest - s/assertAPIEquals/assertAPIErrorCode/

CRM_Utils_REST - Fix PHP 5.4 error

CiviSeleniumSettings - Use and document "siteKey" (instead of "sitekey")

put relationship ahead of account name

CRM-12412 - Implemented unit tests for REST interface authorization

----------------------------------------
* CRM-12412: Implement unit tests for REST interface
  http://issues.civicrm.org/jira/browse/CRM-12412

CRM-12413 - Return an error when api-key does not have an associated CMS user

----------------------------------------
* CRM-12413: CiviCRM does not error on an api-key given through the REST interface that is not connected to a CMS user
  http://issues.civicrm.org/jira/browse/CRM-12413

Merge pull request #567 from dlobo/CRM-12452

CRM-12452

Merge pull request #582 from agh1/financial-types-message-4.3

clarify help text for financial accounts and types (CRM-12473)

Merge pull request #559 from agh1/2nd-membership-warning

Clarify warning for creating duplicate membership

clarify help text for fin accts and types

minor style changes

Style changes

Merge pull request #571 from agh1/bring-back-edit-petition

Edit Petition restored to as it was in 4.2

Merge pull request #575 from eileenmcnaughton/4.3

CRM-12468 preliminary tidy up - whitespace, missing ts

CRM-12378 a new webtest helper function introduction which can help detect break mentioned in CRM-12331

CRM-12468 add pledge next scheduled date filter

CRM-12468 whitespace

CRM-12468 preliminary tidy up - whitespace, missing ts

Merge pull request #574 from twomice/CRM-12460

CRM-12460

Merge pull request #573 from dlobo/CRM-12457

CRM-12457

CRM-12460

Fixed: phpunit doesn't get mysql password from civicrm.settings.local.php

----------------------------------------
* CRM-12460: phpunit doesn't get mysql password from civicrm.settings.local.php
  http://issues.civicrm.org/jira/browse/CRM-12460

CRM-12457

----------------------------------------
* CRM-12457: Typo - 'Succesful' is spelled 'Successful'
  http://issues.civicrm.org/jira/browse/CRM-12457

Merge pull request #561 from totten/upgrade-finish-reload

CRM-12371 - Don't show errors if admin refreshes post-upgrade screen.

Merge pull request #572 from totten/hook-errors-redux

CRM-12370 - Fix another regression from new hook code

CRM-12370 - Code style

----------------------------------------
* CRM-12370: CRM_Utils_Hook::requireCiviModules() assigns infinite weight to extensions
  http://issues.civicrm.org/jira/browse/CRM-12370

CRM-12370 - Reapply 2991458

See also: #552, #560, #570

----------------------------------------
* CRM-12370: CRM_Utils_Hook::requireCiviModules() assigns infinite weight to extensions
  http://issues.civicrm.org/jira/browse/CRM-12370

Edit Petition restored to as it was in 4.2

Merge pull request #555 from totten/givi-review-urls

givi - When pasting in PR URLs for review, accept more URLs

Merge pull request #557 from colemanw/caseCustom

Fix custom data popup CRM-12403

Merge pull request #563 from pratik-joshi/CRM-12404-part-1-fix

fix for CRM-12404 : "Supporter profile is not set for this Personal Camp...

Merge pull request #564 from deepak-srivastava/hulk

Fix for CRM-12428

Merge pull request #565 from deepak-srivastava/ninja

Fix for CRM-12431

Merge pull request #566 from dlobo/CRM-12449

CRM-12449

Merge pull request #569 from yashodha/4.3

CRM-12426

Merge pull request #570 from dlobo/CRM-12416

CRM-12416 also fix warnings when running regen

CRM-12416 also fix warnings when running regen

----------------------------------------
* CRM-12416: Database problems when installing Danish language as basic data set
  http://issues.civicrm.org/jira/browse/CRM-12416

CRM-12426

CRM-12452

----------------------------------------
* CRM-12452: Cannot record activities for individuals with an apostrophe in their last name
  http://issues.civicrm.org/jira/browse/CRM-12452

CRM-12449

----------------------------------------
* CRM-12449: On the import page, the dedupe rules select isn't using the rules' names
  http://issues.civicrm.org/jira/browse/CRM-12449

Fix for CRM-12431 urls not showing up

Fix for CRM-12428

Fix for CRM-12431

Merge pull request #562 from totten/allcoretables-ignore

gitignore - Hide CRM/Core/AllCoreTables.php under 4.3. (The file isn't u...

add warning on standalone add membership form

gitignore - Hide CRM/Core/AllCoreTables.php under 4.3. (The file isn't used in 4.3, but it comes up when switching back and forth between the 4.3 and 4.4 codebases.)

CRM-12371 - Don't show errors if admin refreshes post-upgrade screen.

Note: It's not possible to display the exact same messages because the session is destroyed as part of the upgrade process.

----------------------------------------
* CRM-12371: Refreshing upgrade finish page causes errors
  http://issues.civicrm.org/jira/browse/CRM-12371

pop up warning when selecting member org

Merge pull request #558 from colemanw/comments

Update code comments

Merge pull request #554 from agh1/advmultiselect-narrow-clarify

add and remove labels for report role advmultiselect buttons

Fn comment

Update code comments

rewrite membership warning text

Fix custom data popup CRM-12403

givi - When pasting in PR URLs for review, accept URLs from the "Commits" and "Files Changed" tabs

add and remove labels for report role advmultiselect buttons

Merge pull request #553 from kurund/CRM-12370

worked on CRM-12370, minor typo

worked on CRM-12370, minor typo

Merge pull request #552 from kurund/CRM-12370

worked on CRM-12370, add civicrm modules after main civicrm drupal modul...

worked on CRM-12370, add civicrm modules after main civicrm drupal module

Merge pull request #549 from dlobo/CRM-12424

CRM-12430

Merge pull request #550 from dlobo/CRM-12430

CRM-12430

Merge pull request #548 from totten/joomla-fmt

CRM_Utils_Hook_Joomla - Formatting

CRM_Utils_Hook_Joomla - Formatting

CRM-12430

----------------------------------------
* CRM-12430: CRM_Mailing_BAO_Mailing:create changes all mailing settings back to default
  http://issues.civicrm.org/jira/browse/CRM-12430

CRM-12430

----------------------------------------
* CRM-12430: CRM_Mailing_BAO_Mailing:create changes all mailing settings back to default
  http://issues.civicrm.org/jira/browse/CRM-12430

fix for CRM-12404 : "Supporter profile is not set for this Personal Campaign Page or the profile is disabled. Please contact the site administrator if you need assistance."

Merge pull request #547 from deepak-srivastava/ninja

CRM-12427 Fixes and patch

CRM-12427 Fixes and patch

Merge pull request #539 from kurund/CRM-12407

CRM-12407

Merge pull request #531 from colemanw/versionCompare

Use php version_compare function instead of our own