Explicit return NULL.

Indentation, whitespace and wrapping.

More @todo

Docblock improvements

Add type conversion to constants, because IDE cannot know for sure they are integer. This might be controversial.

More @todo items, possible bugs.

More docblock stuff.

Add @todo tags in CRM_Utils_Request and CRM_Utils_Type.

More docblock improvements.

Merge branch 'giant-rabbit-angular-tests'

crmApp, crmUtil - Tighten scope of crmApp. Move services to crmUtil. Avoid cyclic dependencies.

All services, routes, controllers, and directives should live in specific
Angular modules.  When it comes time to display a complete web page, these
modules are aggregated to form crmApp.  crmApp is a thin piece of glue whose
sole purpose is aggregation.

crmUtil - Alphabetize services

CRM_Core_Page_Angular - Fix badly merged list of modules.

Also: Alphabetize list so that future issues with modules are easier to recognize.

Merge branch 'angular-tests' of https://github.com/giant-rabbit/civicrm-core into giant-rabbit-angular-tests

Merge pull request #4827 from colemanw/CRM-15759

CRM-15759 - Extend in-place editing to more places

CRM-15759 - Fix crmEditable textarea handling

CRM-15759 - Apply crmEditable behavior based on events instead of a template

crmEditable code cleanup

CRM-15759 - Extend in-place editing to more places

CRM-15759 - support empty select option in crmEditable

Remove unused code

Fix membershipType options lookup and display

CRM-15759 - Improve support for crmEditable select options

Fix incorrect participantStatusClass options lookup

Merge pull request #4828 from colemanw/CRM-15773

CRM-15773 - Use the term Order instead of Weight consistently

CRM-15773 - Use the term Order instead of Weight consistently

Merge pull request #4826 from monishdeb/CRM-15740

CRM-15740 QA fix

CRM-15740 QA fix

----------------------------------------
* CRM-15740: Really long URLs cause SQL issue in a trackable mass mail
  https://issues.civicrm.org/jira/browse/CRM-15740

Merge pull request #4823 from jitendrapurohit/webtestfixes

webtest fixes 4.6

webtest fixes 4.6

Merge pull request #4822 from kurund/indentation-fixes

indentation fixes

indentation fixes

Merge pull request #4817 from williamtheaker/patch-2

Replace wizard-bar radical with heavy checkmark

Merge pull request #4820 from kurund/CRM-15705

CRM-15705

Merge pull request #4821 from jitendrapurohit/webtestcorebugfix

Notification for Updation not shown after Save button is clicked on Mana...

Notification for Updation not shown after Save button is clicked on Manage Event Tab for PCP

Merge pull request #4818 from pratikshad/CRM-15770

CRM-15770 fixes issue of empty date processing into membership page for ...

CRM-15770 fixes issue of empty date processing into membership page for offline

----------------------------------------
* CRM-15770: if $dateString is 'null' make no attempt to format it
  https://issues.civicrm.org/jira/browse/CRM-15770

Replace wizard-bar radical with checkmark

Merge pull request #4684 from eileenmcnaughton/CRM-15683-masger

CRM-15678 hookable invalid page handling

CRM-15683 - CRM_Core_Error::handleUnhandledException - Log double exceptions

CRM-15683 - CRM_Core_Invoke - Cleanup comments. Remove duplicate code.

The catch{} block in _invoke() duplicates the code now in LegacyErrorHandler.

CRM-15683 - Add UnhandledExceptionEvent. Fixup LegacyErrorHandler.

CRM-15683 Tim's proposed changes

remove handling for legacy status bounce as not found anywhere in code

CRM-15678 hookable invalid page handling

Conflicts:
CRM/Contribute/Form/ContributionBase.php

Merge pull request #4815 from rohankatkar/Webtests

Webtests Fixes

Merge pull request #4812 from colemanw/CRM-15495

CRM-15495 - Improve entityRef contact_type filter

Webtests Fixes

Merge pull request #4813 from rohankatkar/CRM_15764

CRM- 15764

CRM-15764. Remove bogus opendir() call.

opendir($cmsIncludePath) appears to have been used as is_dir(), not required since we check file_exists("$cmsIncludePath/file.xxx") and that indicates that the directory exists.

Merge fix - CRM-15764

Improve api.getoptions to return nested groups/tags

CRM-15495 - Improve entityRef contact_type filter

Merge pull request #4809 from totten/master-cs2

Cleanup visibility, final newlines, and CONST

Merge pull request #4804 from atif-shaikh/CRM-15767

CRM-15767 - Back Office "Submit Credit Card Contribution" are not being ...

Merge pull request #4780 from williamtheaker/patch-1

Themed wizard-bar

Fixed drush and node_modules/*.info file conflict

Tim noticed that drush doesn't work when node_modules directory exists.
People say that it's because of the *.info files that some things have
in node_modules and suggested adding a line to the npm postinstall
script to remove  any *.info files from there because they aren't really
needed to run stuff. Because we are already doing brower install in the
postinstall script, I broke the postinstall stuff into a separate
script and put both things in there.

phpcs - Fix error, "Expected 1 newline at end of file; XXX found".

Not used anymore.

I don't think this is used anymore. People just use setup.sh.

Install npm and bower modules in setup.

For the test and demo setups to work with the new npm and bower stuff,
npm install needs to get run. Civi buildkit calls setup.sh, so we'll put
it here for now.

Style - Remove @access

1. These annotations are useful in PHP4-style code.
2. These annotations are useless in PHP5-style code. Either they're redundant with the code, or they contradict the code.
3. In cases where there are contradictions, I don't think there's anyway for us to generally resolve the contradiction except to say, "The code is right."
4. As a developer reading .php files, I look at the code and *never* look at @access.

Style - Remove @public, @private, @protected

1. These annotations are not valid in phpdoc. The actual annotation is "@access (public|private|protected)".
2. These annotations are useful in PHP4-style code.
3. These annotations are useless in PHP5-style code. Either they're redundant with the code, or they contradict the code.

phpcs - Fix error, "Visibility must be declared on method"

Get the distmaker working with npm and bower.

We're going to start distributing the node_modules and bower_modules
using npm and bower now instead of having them in packages because we
feel that those are a better way to manage backages than the packages
repository. So the distmaker needs to copy those directories to the
distribution after running 'npm install' to get the packages downloaded.

Also I needed to put a resolution for the angular version in the
bower.json file because some of the angular plugins were resolving to
angular 1.2.x and we want 1.3.x. I also changed the version numbers to
start at 1.3.8, but allow minor version updates because that's what
we're using.

Fixed mispelling: comparitor -> comparator.

It's nicer to have things spelled correctly.

Use the app version of bower instead of machine version.

Once npm installs bower in node_modules, we should be using that one for
all the bower commands instead of the local one.

Updated crmApi to be latest from Angular.tpl

When I did the last bit of merging, I missed the changes to this
function in Angular.tpl which I've moved here.

phpcs - Fix error, "CONST keyword must be lowercase; expected const but found CONST"

Merge pull request #4808 from colemanw/CRM-15759

CRM-15759 - Improve UX of crmEditable

CRM-15759 - Improve UX of crmEditable

Merge pull request #4802 from jitendrapurohit/webtest4.6

webtest fixes 4.6

Merge pull request #4803 from rohankatkar/Webtests

Webtests Fix

Merge pull request #4806 from civicrm/4.5

Merge branch 4.5 into master

Merge pull request #4805 from colemanw/CRM-15766

CRM-15766 - Fix settings form wrongly adding everything to config

CRM-15766 - Fix settings form wrongly adding everything to config

CRM-15767 - Back Office "Submit Credit Card Contribution" are not being saved.

https://issues.civicrm.org/jira/browse/CRM-15767

Webtests Fix

webtest fixes 4.6

Merge pull request #4790 from mattrenner/CRM-15763

CRM-15763 - Payment Express has deprecated SSLv3

Merge pull request #4800 from jitendrapurohit/CRM-15765upgradechanges

upgrade changes for CRM-15765

Merge pull request #4801 from rohankatkar/Webtests

Webtests fix for Reports

upgrade changes for CRM-15765

Webtests fix for Reports

Merge pull request #4794 from civicrm/CRM-15740

CRM-15740 : increase size for url

Merge pull request #4793 from jsnyder83/master

Added missing Indonesian provinces and revised outdated names

Merge pull request #4796 from jitendrapurohit/webtest4.6

webtest fixes 4.6

Merge pull request #4758 from colemanw/CRM-13672

CRM-13672 - Add alerts for security releases

CRM-13672 - Add unit test coverage for CRM_Utils_VersionCheck

CRM-13672 - Run regen to add new settings to schema

CRM-13672 - Add option to suppress current update alerts

CRM-13672 - Add alerts for security releases

Support the new json output from latest.civicrm.org and display security alerts.
Also adds finer-grained control over how version alerts are displayed.

Code/comment cleanup

Merge remote-tracking branch 'upstream/4.5' into 4.5-master-2014-12-30-00-43-32

Merge pull request #4789 from totten/master-test-tx

tests/phpunit/api/v3 - Call useTransaction() where easy. Remove pointless get_info()/construct().

Merge pull request #4797 from totten/master-cs

Code style

Civi/API - Code style

CRM/Extension - Code style

deleted duplicate lines from failed sync issue

Merge remote-tracking branch 'origin/master'

Conflicts:
xml/templates/civicrm_state_province.tpl