When i checked the recent commits i found this edit in the api.php:

* @param mixed $atoms
* An array of atoms if the provider implements
* hook_scald_add_atom_account_count(), otherwise a single atom.

But in fact the hook is called:
hook_scald_add_atom_count()

Comments

jcisio’s picture

jcisio commented
Status: Active » Fixed

Thanks. Fixed.

gifad’s picture

gifad commented
Priority: Minor » Major
Status: Fixed » Active

another typo in scald.api.php, line #369

function scald_image_scald_add_form_fill(&$atoms, $form, $form_state) {

don't understand why this does not throw a php error...

jcisio’s picture

jcisio commented
Priority: Major » Minor
Status: Active » Fixed

Do you mean "scald_image" should be replaced with "hook"? Fixed it. Also, scald.api.php is documentation and never executed, so you can't see PHP error.

gifad’s picture

gifad commented
Status: Fixed » Active

while this issue is not closed...

I found another one, lines 22..25 :

 * system_modules_submit() [The form at /admin/build/modules is submitted]
 *   -> {provider}_install() [Drupal hook]
 *   -> {provider}_enable() [Drupal hook]
 *   -> {provider}_scald_provider()

The url reminds me Drupal6, and actually searching google for “ {provider}_scald_provider()” points to scald_provider_api.txt

good night (;)

jcisio’s picture

jcisio commented
Status: Active » Fixed

Deleted. It is old doc from D6 when the provider registration was complex and difficult.

Status: Fixed » Closed (fixed)

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