The Charts and Graphs module uses at least two, depending in the interpretation, three different names for itself:

  • Charts and Graphs (used in most places)
  • Charts Core API (used in the main .info file)
  • Charts (used in submodules .info files)

Besides being a natural source of confusion using these names makes things even messier as people end up having difficulty differentiating this module from Chart API and Charts, two similar modules.

Besides that the submodules also don't have standardized naming as both amCharts and Open Charts Flash 2 use the library names as its nice name and Bluff uses Charts: Bluff.

I would like to solve these issues. Here is what I propose:

  1. use Charts and Graphs for the main module name in all places
  2. use only the library name as the nice name for submodules

    3. I will wait a few days so anybody interested can have a say on this, otherwise I will move forward with this next week.

Comments

irakli’s picture

irakli commented

Sounds like a great suggestion.

I assume this can not break dependencies or existing installations of modules that depend on charts_graphs (e.g. Views Charts) because machine names won't change?

Thank you.

rsevero’s picture

rsevero commented

Yes, machine names won't change by this issue.

I would like to change submodule machine names from charts_... to charts_graphs_... but that's an issue for another issue ;)

I will make the changes related to this issue.

rsevero’s picture

rsevero commented
Status: Active » Fixed

Fixed with commit #384558.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.