Skip to content
This repository has been archived by the owner on Nov 7, 2019. It is now read-only.

Fix for infinite recursion bug #28

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions jquery.flot.axislabels.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,13 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
*/

(function ($) {
var options = {
var jqueryFlotAxisLabelsIsRegistered = false;
var options = {
axisLabels: {
show: true
}
};

function canvasSupported() {
return !!document.createElement('canvas').getContext;
}
Expand Down Expand Up @@ -454,8 +455,8 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
}


if(jqueryFlotAxisLabelsIsRegistered !== undefined){
var jqueryFlotAxisLabelsIsRegistered = true;
if(jqueryFlotAxisLabelsIsRegistered !== false){
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be === false.

jqueryFlotAxisLabelsIsRegistered = true;
$.plot.plugins.push({
init: init,
options: options,
Expand Down