There is a namespace problem here on 2 fronts.

1) This is toolbar module and so it the toolbar in D7. Discussing the two can easily become confusing.

2) This toolbar.module cannot run alongside D7 toolbar module. There is a namespace conflict. If this module is ported to D7 and even exists in a modules folder the core toolbar module will no longer be accessible to use.

Using the same namespace of a core module is a bad idea.

Comments

lelizondo’s picture

lelizondo commented
Status: Active » Fixed

This module is *not* going to be ported to D7, since the D7 Toolbar module is so much better. Having said that, I don't know how discussing the two can become confusing. They don't share issues and they don't share the same major version, they don't share anything but the idea and the name.

It has been a good practice (see vertical_tabs) to backport some of the modules and functionality in new drupal versions. That's the main idea behind this module, I don't think the drupal dev team will backport the toolbar module and the shortcuts module, that's why I did it since I needed them.

I honestly (and I'm not trying to defend my decision), don't think there's a problem, look at the D6 RDF module, there's also an RDF module with D7 in core. The D6 RDF will have a D7 version, and obviously, they changed the module's name (http://github.com/scor/rdfx) to rdfui, but the project remains the same for D6 and D7.

Anyway, if the security team, the D7 developers team, or the cvs team wants to contact me and tell me to change the name of this module, I'll do it without a problem.

Status: Fixed » Closed (fixed)

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