[Commits] [fluid-project/infusion] 0c7dd4: FLUID-4915: args defaults to [] for apply

GitHub noreply at github.com
Tue Feb 12 11:30:18 EST 2013


  Branch: refs/heads/master
  Home:   https://github.com/fluid-project/infusion
  Commit: 0c7dd4e82b539a59b64d1107abeceb6cafb7fbc4
      https://github.com/fluid-project/infusion/commit/0c7dd4e82b539a59b64d1107abeceb6cafb7fbc4
  Author: Justin Obara <obarajustin at gmail.com>
  Date:   2013-02-06 (Wed, 06 Feb 2013)

  Changed paths:
    M src/webapp/framework/core/js/Fluid.js
    M src/webapp/tests/framework-tests/core/js/FluidJSTests.js

  Log Message:
  -----------
  FLUID-4915: args defaults to [] for apply

Changed fluid.invokeGlobalFunction to set the args to default to an empty array "[]" when args are falsey. This fixes an issue with IE8. I've also added a couple of unit tests in FLuidJSTests.js to verify the behaviour.


  Commit: 2740d9784eedbdcc9c5a9bec253bddcebb1f138f
      https://github.com/fluid-project/infusion/commit/2740d9784eedbdcc9c5a9bec253bddcebb1f138f
  Author: Justin Obara <obarajustin at gmail.com>
  Date:   2013-02-06 (Wed, 06 Feb 2013)

  Changed paths:
    M src/webapp/tests/framework-tests/core/js/FluidJSTests.js

  Log Message:
  -----------
  FLUID-4915: linting

Needed to lint for the test file to even run in IE6. The test I added passes, but there are some others that fail.


  Commit: 0e2814890cadfedc4d9bff72896f1a2cb5775f92
      https://github.com/fluid-project/infusion/commit/0e2814890cadfedc4d9bff72896f1a2cb5775f92
  Author: Michelle D'Souza <michelled33 at gmail.com>
  Date:   2013-02-12 (Tue, 12 Feb 2013)

  Changed paths:
    M src/webapp/framework/core/js/Fluid.js
    M src/webapp/tests/framework-tests/core/js/FluidJSTests.js

  Log Message:
  -----------
  Merge remote-tracking branch 'jobara/FLUID-4915'

* jobara/FLUID-4915:
  FLUID-4915: linting
  FLUID-4915: args defaults to [] for apply


Compare: https://github.com/fluid-project/infusion/compare/8a0ebfa25065...0e2814890cad


More information about the Commits mailing list