While looking over the code I've noticed that there is some boilerplate code for generating unique html ids. There is drupal_html_id() that does exactly that. We should use it.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

torotil’s picture

Status: Active » Needs review
FileSize
767 bytes

Here is a patch doing exactly that.

Also you might want to replace all the tabs with spaces to conform to drupal coding standards. I'm happy to provide a patch for that too.

  • interdruper committed 9f50437 on 7.x-1.x authored by torotil
    Issue #2468489 by torotil: Use drupal_html_id instead of custom code.
    
interdruper’s picture

Status: Needs review » Closed (fixed)