Hi,

I lost 3 hours in trying to install zoomify.

I installed a clean drupal 6 and after seeing that the README is not quite good ... I finally give up.

Furthermore in the code I saw 2 mistakes so far:
1. the zoomify tab was not showing at all cause you have :

/**
* Implementation of hook_menu().
*/
function zoomify_menu() {
...
$items['node/%zoomify/zoomify'] = array(

instead of $items['node/%/zoomify'] = array(

after I have corrected it the menu item (=tab) showed up.

and 2. the menu callback from above:

'page callback' => 'zoomify_display',

gets arg1 which is a "node ID"

so, the function should be :

function zoomify_display($nid) {
$node = node_load($nid);
return theme_zoomify_page($node);
}

I guess when you were releasing for drupal 6 you did not test at all, right??

Also since the code is not well documented (at least a brief explanation of what is going on. ..) I cannot help you a lot with patches...

I am trying with the php image processor and with the Mac version of the zoomify files from the zoomify site...

In the readme it is not very clear which package to download and which files from there to use and where to put them....

Please give me detailed instructions how to install - the module is uninstallable for me - hence I filed a bug report.

Comments

rankinstudio’s picture

Yea, enabling this just crashes my site...

hmm

infojunkie’s picture

@rsvelko: you are mistaken in your analysis of the hook_menu code. What is written in the module is intentional and functions correctly on D6, which is my daily testing environment. Zoomify does not show the tab when there's no image to display. Find out more about wildcard usage here: http://drupal.org/node/209056.

Installation instructions are available on the module's main page: http://drupal.org/project/zoomify. Please let me know if anything's unclear.

@rankinstudio: would you care to elaborate on the circumstances of the crash? Otherwise, I can't fix the problem. Thanks.

rsvelko’s picture

agree on the wildcard thing.

Will try and install these days again.

I propose you run through the instructions on the project page and the README once again and readm them imagining you are a dumb user. Istrongly suspect that there is a complexity wall in the moment for regular users trying to install the module...

More later.

infojunkie’s picture

Status: Active » Fixed

Fixed unless I hear back from the original poster.

Status: Fixed » Closed (fixed)

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