This is a nice module with great features. But could it support more of installation profile. My experience using it shows the following error on Drupal installation page;

    * Money CCK field requires, at least, version 6.x-1.1 of the Format Number API module.
    * Money CCK field requires, at least, version 6.x-1.0 of the Formatted Number CCK module.

I already have updated versions of the format number api and formatted number cck modules.

My findings shows that nothing is checking the versions of the format number api and formatted number cck modules before an error is thrown. Please correct me if i am wrong.

Thanks

Comments

markus_petrux’s picture

markus_petrux commented
Priority: Critical » Normal
Status: Active » Closed (works as designed)

The problem is that drupal_load() in hook_requirement('install') returns FALSE. It is unable to load the required modules because the {system} table, where information about modules is stored by Drupal, is not ready yet when drupal_load() invokes drupal_get_filename(), if that happens from an installation profile. It works as it should, however, if Money is installed manually after the required modules.

This is unexpected behavior of drupal_load(), and there's no other documented way to do this kind of checks in hook_requirement('install'), AFAICT, so this is a Drupal core bug. I do not know where it's been reported, though.

As a developer, I do not want to let you install Money module, if all requirements are not met, and that should be done in hook_requirement('install'). Otherwise, if this check is not done, and you install Money without the other modules, then it would be more complex to ensure the module does not break anything in your site.

drecute’s picture

drecute commented

Thank you. I though as much. It means this module can't be included in an installation profile because this function is not part of the functions that is available to the hook system at this stage. i understand though.
Thanks.