Skip to content

Commit

Permalink
fix(clean-up): Remove swfobject and other unused scripts and pages (#343
Browse files Browse the repository at this point in the history
)

`swfobject` was used by our player, for when audio streams were still still delivered using Adobe/Macromedia Flash. Fortunately for us all, we got rid of this ancient technology! 🧹
  • Loading branch information
adrienjoly committed Aug 29, 2020
1 parent 2260721 commit fb94438
Show file tree
Hide file tree
Showing 11 changed files with 17 additions and 2,158 deletions.
37 changes: 17 additions & 20 deletions .codeclimate.yml
Original file line number Diff line number Diff line change
@@ -1,33 +1,30 @@
engines:
engines:
duplication:
enabled: true
config:
languages:
- javascript
- javascript
eslint:
enabled: true
channel: "eslint-2"
channel: 'eslint-2'
config:
config: ./.eslintrc

ratings:
paths:
- "./app/**.js"
- './app/**.js'

exclude_patterns:
- "./config/"
- "./public/js/jquery*.js"
- "./public/js/swfobject.js"
- "./public/js/soundmanager2.js"
- "./public/js/*-all.js"
- "./public/js/*-min.js"
- "./public/js/*.min.js"
- "features/"
- "**/node_modules/"
- "**/unused/"
- "**/unused/"
- "**/spec/"
- "**/test/"
- "**/tests/"
- "**/vendor/"
- "**/*.d.ts"
- './config/'
- './public/js/jquery*.js'
- './public/js/soundmanager2.js'
- './public/js/*-all.js'
- './public/js/*-min.js'
- './public/js/*.min.js'
- 'features/'
- '**/node_modules/'
- '**/spec/'
- '**/test/'
- '**/tests/'
- '**/vendor/'
- '**/*.d.ts'
1 change: 0 additions & 1 deletion app/templates/feed.js
Original file line number Diff line number Diff line change
Expand Up @@ -208,7 +208,6 @@ exports.renderFeedEmbed = function (feedHtml, options) {
? 'feedEmbedV2.css'
: 'feedEmbed.css'
);
options.js.push('swfobject.js');
options.js.push('playem-min.js');
options.js.push('playem-youtube-iframe-patch.js');
options.js.push('whydPlayer.js');
Expand Down
3 changes: 0 additions & 3 deletions app/templates/mainTemplate.js
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,6 @@ exports.renderWhydPage = function (params) {
'ui.js',
'whyd.js', // topicBrowser.js
// "ContentEmbed.js", // definitely replaced by playemjs, at last! :-)
'swfobject.js',
]
)
.concat(['playem-' + playemFile + '.js'])
Expand Down Expand Up @@ -397,7 +396,6 @@ exports.renderWhydPage = function (params) {
'<div class="topWarning">Warning: your web browser is not supported by Openwhyd. Please upgrade to a modern browser.</div>',
'<![endif]-->',
// '<a id="feedbackLink" href="mailto:contact@openwhyd.org?subject=[proto-support]&body=Please%20enter%20your%20feedback%20here">Send feedback</a>',
// '<a id="feedbackLink" href="#" onclick="UserVoice.Popin.show(uservoiceOptions); return false;">&nbsp;</a>',
exports.renderHeader(user, params.whydHeaderContent, params),
'<div id="contentPane">',
' <div id="mainPanel">',
Expand All @@ -408,7 +406,6 @@ exports.renderWhydPage = function (params) {
params.footer || exports.footer,
exports.olark,
params.endOfBody || '',
// '<script src="'+render.urlPrefix+'/js/uservoice.js"></script>'
];

return this.renderWhydFrame(out.join('\n'), params);
Expand Down
13 changes: 0 additions & 13 deletions public/html/YoutubePlayerIframe-local.html

This file was deleted.

56 changes: 0 additions & 56 deletions public/html/testDeezerFallback.html

This file was deleted.

Loading

0 comments on commit fb94438

Please sign in to comment.