Implementing hook_requirements will let us assure we have xapian php binding on the server, an abort if we have not.

CommentFileSizeAuthor
#2 fix-hook_requirements.patch5.44 KBmarvil07

Comments

marvil07’s picture

Title: implement hook_requirements » move hook_requirements to install file

from hook_requirements api documentation:

Note that this hook, like all others dealing with installation and updates, must reside in a module_name.install file, or it will not properly abort the installation of the module if a critical requirement is missing.

marvil07’s picture

Title: move hook_requirements to install file » fix hook_requirements()
Status: Active » Needs review
Issue tags: +installation requirement
StatusFileSize
new5.44 KB

we need to:

- move it to install file
- use get_t() for translations
- show a more descriptive message when xapian bindings are not installed, by adding a "you need to install xapian bindings" in addition to the real include error

marvil07’s picture

Category: feature » bug
marvil07’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)
Issue tags: -installation requirement

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