This issue seems very similar to #362889: Move drupal_common_theme() from common.inc into theme.inc

Problem/Motivation

Theme development for drupal is too complicated, and people often don't know where to find the code they are looking for.

Proposed resolution

In order to make the drupal theme process "make more sense" we should move theme functions to where they expect to be located.

Remaining tasks

- move all system module's theme functions back to the system module (system.theme.inc? system.module? ideas?)
- if these theme functions don't "belong" to system module, move them to theme.inc

API changes

?

Comments

bleen’s picture

bleen commented

has this already been completed? ... I dont see any theme functions in common.inc any longer

fixed?

c4rl’s picture

c4rl commented

Yeah, I'm not sure I understand this one. AFAIK, there haven't been any theme functions in common.inc in D7 or even D6.

@jenlampton, can you clarify?

c4rl’s picture

c4rl commented
Priority: Normal » Major
Status: Active » Postponed (maintainer needs more info)
Issue tags: +consistency, +theme system cleanup

@jenlampton Do you instead mean theme.inc? Can you provide some examples?

jenlampton’s picture

jenlampton
she / her
commented
Status: Postponed (maintainer needs more info) » Closed (duplicate)

Sorry guys, this is duplicate of #362889: Move drupal_common_theme() from common.inc into theme.inc which is now fixed :)

jenlampton’s picture

jenlampton
she / her
commented
Issue summary: View changes

add link to other issue