areski / django-nvd3

Django wrapper for nvd3 - It's time for beautiful charts
Other
416 stars 124 forks source link

unslugified chartname in include_container #56

Open ingodjango opened 9 years ago

ingodjango commented 9 years ago

Hi All, I have an issues with version 0.13.7.

First is in NVD3Chart.py line 173-175: (from python-nvd3)

def _slugify_name(self, name):
    """Slufigy name with underscore"""
    self.name = slugify(name, separator='_')

I have a Container name that looks like 'ContainerName' in my template, also do include_container with that name. Since slugify removes the capital letters d3.select will not be able to find the correct container. In content.html in line 101: d3.select('#{{ chart.name }} svg') Will then look for 'containername svg' instead of 'ContainerName svg' . The reason seems to be that in nvd3_tags in line 94 the chart name is an unslugified representation: chart.name = str(include_container) using: chart.name = slugify(includecontainer, separator='') seems to work.