FLUID-5264: progressiveCheckerForComponent does not look for progressiveChecker options supplied to a component at instantiation.

Description

Currently the progressiveCheckerForComponent looks up a components defaults when attempting to find the the progressiveCheckerOptions.

https://github.com/fluid-project/infusion/blob/master/src/framework/enhancement/js/ProgressiveEnhancement.js#L130

However, this ignores any options supplied to component at instantiation time. Amongst other things, this prevents an integrator from directly adding a progressive enhancement step to a component at instantiation.

Comments

• Justin Obara commented 2014-01-31T13:19:52.049-0500

  Submitted a pull request with a unit test for the issue
  https://github.com/fluid-project/infusion/pull/460

• Antranig Basman commented 2014-03-18T21:50:49.236-0400

  I tried making a simpleminded fix for this issue based on "fluid.getForComponent" but discovered that our usage pattern in the Uploader is too complex to be cleared up without major work. The issue is that the uploader is created in two varieties of configuration, one where the progressiveChecker is attached to the component as "uploaderContext" and another where it is attached to the "staticEnvironment". All of these areas of the framework are in the process of being rethought, together with JIRAs such as http://issues.fluidproject.org/browse/FLUID-5249 and http://issues.fluidproject.org/browse/FLUID-5241 - so it would be best to rework this implementation thoroughly - at the same time renaming it as "contextAwareness" or so.

• Justin Obara commented 2015-06-15T14:28:57.422-0400

  pull request: https://github.com/fluid-project/infusion/pull/591

• Antranig Basman commented 2015-08-20T10:01:35.270-0400

  Merged into trunk at revision 282f1a318718eed0b0ec060fb8b4ad254417fd7e

  progressiveChecker is abolished and replaced by contextAwareness