Ticket #2531319 (closed defect)

Reporter


Standa Opichal
Opened: 10/21/11
Last modified: 05/10/12
Status: closed
Type: defect
Resolution: fixed

Owner


Dav Glass
Target Release: 3.5.0
Priority: P3 (normal)
Summary: The aliased modules are reported as missing
Description:

This causes a Y.Loader or Y.Get (if present) to trigger even when all the real modules where found with attempts to load e.g. 'attribute' (alias to attribute-base and attribute-complex).

Type: defect Observed in Version: 3.4.1
Component: Loader Severity: S3 (normal)
Assigned To: Dav Glass Target Release: 3.5.0
Location: Library Code Priority: P3 (normal)
Tags: Relates To:
Browsers: N/A
URL:
Test Information:

Change History

Standa Opichal

Posted: 10/21/11

Standa Opichal

Posted: 10/21/11
  • component changed from None to Loader
  • owner changed from Jenny Donnelly to Dav Glass
  • status changed from new to assigned

Dav Glass

Posted: 11/8/11
  • location changed to Library Code
  • milestone changed to 3.5.0
  • priority changed to P3 (normal)
  • status changed from assigned to accepted

Dav Glass

Posted: 11/8/11
  • estimated changed from 0 to 0.1
  • remaining changed from 0 to 0.1
  • sprint changed to sprint 1

Dav Glass

Posted: 11/10/11
  • estimated changed from 0.1 to 0
  • remaining changed from 0.1 to 0
  • sprint changed from sprint 1
  • status changed from accepted to infoneeded

Do you have a repro case for this and a test?

I applied the patch, but my tests failed afterward so I needed to revert it.

Dav Glass

Posted: 11/10/11
  • resolution changed to fixed
  • status changed from infoneeded to checkedin

Fixes #2531319 - Patch from opichals for Alias modules
View Commit: 0219aaded8ce350e36b02d364e533cbef2afe9e6

Dav Glass

Posted: 11/10/11
  • milestone changed from 3.5.0
  • resolution changed from fixed
  • status changed from checkedin to reopened

This was reverted..

Dav Glass

Posted: 11/10/11
  • status changed from reopened to infoneeded

Dav Glass

Posted: 11/18/11
  • resolution changed to fixed
  • status changed from infoneeded to checkedin

Fixes #2531319 - Patch from opichals for Alias modules
View Commit: 0219aaded8ce350e36b02d364e533cbef2afe9e6

Standa Opichal

Posted: 11/19/11
  • resolution changed from fixed
  • status changed from checkedin to reopened

Sorry Dav, I am late with the repro stuff... here you go https://gist.github.com/1379337

Dav Glass

Posted: 11/28/11
  • milestone changed to 3.5.0

Dav Glass

Posted: 11/28/11
  • estimated changed from 0 to 0.1
  • remaining changed from 0 to 0.1
  • sprint changed to sprint 1

Dav Glass

Posted: 11/28/11
  • status changed from reopened to accepted

Dav Glass

Posted: 11/28/11

Added a fix for this, but there was a slight change in your test case. Here's an updated test case:

https://gist.github.com/1400882

First, setting debug: true will auto include dump and log. So I removed that.
Then setting lang to '' keeps the intl module from loading.

Then event-custom requires oop, so I added that to the dep list.

Then it showed the behavior of fetching Loader for the 2 aliases.

Dav Glass

Posted: 11/28/11
  • completed changed from 0 to 0.1
  • remaining changed from 0.1 to 0

Dav Glass

Posted: 11/28/11
  • resolution changed to fixed
  • status changed from accepted to checkedin

Fixes #2531319 - Added a proper fix for this issue
View Commit: f96ccb29875f22e2f23c811712d3a27d527ec4c5

Dav Glass

Posted: 12/6/11

Fixes #2531319 - Added a proper fix for this issue
View Commit: f96ccb29875f22e2f23c811712d3a27d527ec4c5

Jenny Donnelly

YUI Developer

Posted: 05/10/12
  • status changed from checkedin to closed

checkedin -> closed