Fixes #1451, fixed bug where detect web triggered on private groups

This commit is contained in:
Matt Burton 2009-05-20 21:04:39 +00:00
parent afb950ba3a
commit 859b1df6c9

View file

@ -8,7 +8,7 @@
"maxVersion":"", "maxVersion":"",
"priority":100, "priority":100,
"inRepository":true, "inRepository":true,
"lastUpdated":"2009-05-14 20:55:00" "lastUpdated":"2009-05-20 20:55:00"
} }
function detectWeb(doc, url) { function detectWeb(doc, url) {
@ -16,6 +16,12 @@ function detectWeb(doc, url) {
var nsResolver = namespace ? function(prefix) { var nsResolver = namespace ? function(prefix) {
if (prefix == 'x') return namespace; else return null; if (prefix == 'x') return namespace; else return null;
} : null; } : null;
// don't display for private groups
if (url.match(/\/groups\/[0-9]+\/items/)) {
return false;
}
var a = doc.evaluate('//li[@id="library-tab"]/a[text()="My Library"]', doc, nsResolver, XPathResult.ANY_TYPE, null).iterateNext(); var a = doc.evaluate('//li[@id="library-tab"]/a[text()="My Library"]', doc, nsResolver, XPathResult.ANY_TYPE, null).iterateNext();
// Skip current user's library // Skip current user's library
if (a && url.indexOf(a.href.match(/(^.+)\/items/)[1]) == 0) { if (a && url.indexOf(a.href.match(/(^.+)\/items/)[1]) == 0) {
@ -28,6 +34,7 @@ function detectWeb(doc, url) {
} }
return "multiple"; return "multiple";
} }
// Individual item // Individual item
else if (url.match(/\/items\/[0-9]+(\?.*)?$/)) { else if (url.match(/\/items\/[0-9]+(\?.*)?$/)) {
// TODO: embed in page, because this is absurd // TODO: embed in page, because this is absurd