Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

Commit

Permalink
fix bz 5472979 server resources in build
Browse files Browse the repository at this point in the history
server affinity resources do not need to be uri
addressable. with this change store.getAllUrls()
will no longer include foo.server.js resources,
and hence be omitted from `mojito build`
  • Loading branch information
Isao Yagi committed Oct 18, 2012
1 parent 5a83c42 commit 3f6aeaa
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions lib/app/addons/rs/url.server.js
Original file line number Diff line number Diff line change
Expand Up @@ -181,8 +181,9 @@ YUI.add('addon-rs-url', function(Y, NAME) {
rollupParts = [],
rollupFsPath;

// Don't clobber a URL calculated by another RS addon.
if (res.hasOwnProperty('url')) {
// Don't clobber a URL calculated by another RS addon, or bother to
// proceed for server affinity resources that don't need uris
if (res.hasOwnProperty('url') || ('server' === res.affinity)) {
return;
}

Expand Down

0 comments on commit 3f6aeaa

Please sign in to comment.