Hi Edvaldo Biancarelli,
Below are the things i observed...
podlove_player.ins...

Contribution Date
Technology
Contribution Details

Hi Edvaldo Biancarelli,

Below are the things i observed...

podlove_player.install :
1) This is not an application blocker but you should not setup variables inside hook_install() . Drupal by default provides you the ability to use default values inside variable_get() if they are not set.
Refer the variable_get() documentation & the 'default' parameter in it.

2) Suggestion : I guess this message

drupal_set_message($t('Podlove Player Module was successfully installed. Do not forget to install Podlove Library. See Podlove Help Page in <a href="@link">Administration > Help > Podlove Player</a>.' , array('@link' => '/admin/help/podlove_player') ));

should be inside hook_enable().

podlove_player.install
1)You are using system_settings_form(), so no need to setup variables actually in install file, just use the default parameter inside variable_set() in your "#default_value".

podlove_player.module
1) $return = 'Library Podlove Web Player not found'; The string should be inside t() function.

Contribution Author
Files count
0
Patches count
0