(#17522) Only print load errors to debug with non-specific context #1280

Closed
wants to merge 1 commit into
from

Projects

None yet

1 participant

Member

Fixes a variable assignment that landed in the wrong branch during a conflict
merge (a4451fb). Commit a8575f4 fixed a spec failure but the original code
wasn't restored.

This restores the original functionality in #14136 where a non-specific
context (i.e. /files/etc) would be optimised (only load lenses for files under
/etc) but since it was vague, would only log load errors to debug. Instead,
load errors were reported as warnings for non-specific contexts.

Load errors will continue to be reported as warnings for supplied contexts if
they're specific.

@domcleal domcleal (#17522) Only print load errors to debug with non-specific context
Fixes a variable assignment that landed in the wrong branch during a conflict
merge (a4451fb).  Commit a8575f4 fixed a spec failure but the original code
wasn't restored.

This restores the original functionality in #14136 where a non-specific
context (i.e. /files/etc) would be optimised (only load lenses for files under
/etc) but since it was vague, would only log load errors to debug.  Instead,
load errors were reported as warnings for non-specific contexts.

Load errors will continue to be reported as warnings for supplied contexts if
they're specific.
3e460ba
Member

See 911e2d4 for the original code for #14136.

@domcleal domcleal closed this Feb 6, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment