diff --git a/modules/media_internet/media_internet.module b/modules/media_internet/media_internet.module
index 9a01e92..137264c 100644
--- a/modules/media_internet/media_internet.module
+++ b/modules/media_internet/media_internet.module
@@ -79,24 +79,30 @@ function media_internet_add($form, &$form_state = array(), $types = NULL) {
$form['#validators']['media_file_validate_types'] = array($types);
}
- $form['providers'] = array();
- $form['providers']['header'] = array('#markup' => '
' . t('Supported Providers') . '
');
+ $items = array();
+
foreach (media_internet_get_providers() as $key => $provider) {
if (empty($provider['hidden']) || $provider['hidden'] != TRUE) {
- if (isset($provider['image'])) {
- $form['providers'][$key] = array('#markup' => theme('image', array('path' => $provider['image'], 'title' => $provider['title'])));
- }
- else {
- $form['providers'][$key] = array('#markup' => $provider['title']);
- }
- // Wrap the provider in a div so we can make a nice list
- $form['providers'][$key]['#prefix'] = '';
- $form['providers'][$key]['#suffix'] = '
';
+ $items[] = $provider['title'];
}
}
- if (count($form['providers']) == 1) {
- // Just the header, no actual providers
- unset($form['providers']['header']);
+
+ if (!empty($items)) {
+ $form['providers'] = array();
+ $form['providers']['header'] = array(
+ '#markup' => '' . t('Supported Providers') . '
',
+ );
+
+ $variables = array(
+ 'items' => $items,
+ 'title' => NULL,
+ 'type' => 'ul',
+ 'attributes' => array(),
+ );
+
+ $form['providers']['provider_list'] = array(
+ '#markup' => theme('item_list', $variables),
+ );
}
$form['actions'] = array('#type' => 'actions');