Moved SVGSupport call to use OC.Util.SVGSupport()

This commit is contained in:
Vincent Petry 2014-04-09 15:23:09 +02:00
parent d8f56e3c00
commit f84d66a24f
2 changed files with 6 additions and 6 deletions

View File

@ -295,7 +295,7 @@ var OC={
*/ */
imagePath:function(app,file){ imagePath:function(app,file){
if(file.indexOf('.')==-1){//if no extension is given, use png or svg depending on browser support if(file.indexOf('.')==-1){//if no extension is given, use png or svg depending on browser support
file+=(SVGSupport())?'.svg':'.png'; file+=(OC.Util.hasSVGSupport())?'.svg':'.png';
} }
return OC.filePath(app,'img',file); return OC.filePath(app,'img',file);
}, },
@ -497,7 +497,7 @@ var OC={
throw e; throw e;
}); });
} }
if(!SVGSupport()) { if(!OC.Util.hasSVGSupport()) {
OC.Util.replaceSVG(); OC.Util.replaceSVG();
} }
}).show(); }).show();
@ -874,7 +874,7 @@ function initCore() {
initSessionHeartBeat(); initSessionHeartBeat();
} }
if(!SVGSupport()){ //replace all svg images with png images for browser that dont support svg if(!OC.Util.hasSVGSupport()){ //replace all svg images with png images for browser that dont support svg
OC.Util.replaceSVG(); OC.Util.replaceSVG();
}else{ }else{
SVGSupport.checkMimeType(); SVGSupport.checkMimeType();

View File

@ -147,19 +147,19 @@ describe('Core base tests', function() {
}); });
describe('Images', function() { describe('Images', function() {
it('Generates image path with given extension', function() { it('Generates image path with given extension', function() {
var svgSupportStub = sinon.stub(window, 'SVGSupport', function() { return true; }); var svgSupportStub = sinon.stub(OC.Util, 'hasSVGSupport', function() { return true; });
expect(OC.imagePath('core', 'somefile.jpg')).toEqual(OC.webroot + '/core/img/somefile.jpg'); expect(OC.imagePath('core', 'somefile.jpg')).toEqual(OC.webroot + '/core/img/somefile.jpg');
expect(OC.imagePath(TESTAPP, 'somefile.jpg')).toEqual(TESTAPP_ROOT + '/img/somefile.jpg'); expect(OC.imagePath(TESTAPP, 'somefile.jpg')).toEqual(TESTAPP_ROOT + '/img/somefile.jpg');
svgSupportStub.restore(); svgSupportStub.restore();
}); });
it('Generates image path with svg extension when svg support exists', function() { it('Generates image path with svg extension when svg support exists', function() {
var svgSupportStub = sinon.stub(window, 'SVGSupport', function() { return true; }); var svgSupportStub = sinon.stub(OC.Util, 'hasSVGSupport', function() { return true; });
expect(OC.imagePath('core', 'somefile')).toEqual(OC.webroot + '/core/img/somefile.svg'); expect(OC.imagePath('core', 'somefile')).toEqual(OC.webroot + '/core/img/somefile.svg');
expect(OC.imagePath(TESTAPP, 'somefile')).toEqual(TESTAPP_ROOT + '/img/somefile.svg'); expect(OC.imagePath(TESTAPP, 'somefile')).toEqual(TESTAPP_ROOT + '/img/somefile.svg');
svgSupportStub.restore(); svgSupportStub.restore();
}); });
it('Generates image path with png ext when svg support is not available', function() { it('Generates image path with png ext when svg support is not available', function() {
var svgSupportStub = sinon.stub(window, 'SVGSupport', function() { return false; }); var svgSupportStub = sinon.stub(OC.Util, 'hasSVGSupport', function() { return false; });
expect(OC.imagePath('core', 'somefile')).toEqual(OC.webroot + '/core/img/somefile.png'); expect(OC.imagePath('core', 'somefile')).toEqual(OC.webroot + '/core/img/somefile.png');
expect(OC.imagePath(TESTAPP, 'somefile')).toEqual(TESTAPP_ROOT + '/img/somefile.png'); expect(OC.imagePath(TESTAPP, 'somefile')).toEqual(TESTAPP_ROOT + '/img/somefile.png');
svgSupportStub.restore(); svgSupportStub.restore();