haskal
/
jiti-meet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Prefers public getters over duplication of logic and/or access to private fields at multiple locations.

This commit is contained in:
Lyubomir Marinov 2016-01-13 22:00:30 +02:00
parent d7c9a97e8d
commit 2b570a2251
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
modules/UI/Feedback.js
View File

@ -79,7 +79,7 @@ var Feedback = {
init: function () {
// CallStats is the way we send feedback, so we don't have to initialise
// if callstats isn't enabled.
if (!callStats.isEnabled())
if (!this.isEnabled())
return;
$("div.feedbackButton").css("display", "block");
@ -93,6 +93,9 @@ var Feedback = {
* @return true if the feedback functionality is enabled, false otherwise.
*/
isEnabled: function() {
// XXX callStats.isEnabled() indicates whether we are allowed to attempt
// to integrate callstats.io. Whether our attempt was/is/will be
// successful is a different question.
return callStats.isEnabled();
},
/**
@ -234,4 +237,4 @@ var Feedback = {
};
// Exports the Feedback class.
module.exports = Feedback;
module.exports = Feedback;

4
modules/statistics/CallStats.js
View File

@ -31,7 +31,7 @@ var callStatsIntegrationEnabled = config.callStatsID && config.callStatsSecret;
var CallStats = {
init: function (jingleSession) {
if(!callStatsIntegrationEnabled || callStats !== null) {
if(!this.isEnabled() || callStats !== null) {
return;
}
@ -146,7 +146,7 @@ var CallStats = {
_reportError: function (type, e, pc) {
if (callStats) {
callStats.reportError(pc, this.confID, type, e);
} else if (callStatsIntegrationEnabled) {
} else if (this.isEnabled()) {
pendingErrors.push({
type: type,
error: e,