Support/Comments for Doc Finder Module

Hello Paul,

thanks for taking my concerns regarding the events further. I guess its only the naming of the events that is a little confusing. However I really would like to have the opinion of a core developer on this topic before I start implementing this into Doc Finder.

So lets the what new information the new thread brings up.

Hello Paul,

so I guess I will just strip the event code out of the Doc Finder source? As I understood your discussion with Jason about this matter the related events should not be called if not related to actions in the manager, am I right?

Bogdan

My opinion is still that the events should be called, or at least that they *can* be called by setting a parameter (e.g. $callEvents = 1;). I’m not sure that Jason gave a definitive answer one way or the other, so you might want to ask him directly. It sounds like the future Revolution code may have other ways of accomplishing the same goal, but there would still need to be an option in your code somewhere to take advantage of those methods.

I loaded the latest version and when I want to search and replace something, I get the following error:

Fatal error: Call to undefined function: str_ireplace() in /home/mywebsite/www/mysite.com/assets/modules/docfinder/functions.php on line 722

Any suggestions? Thanks!

Yes, Doc Finder requires PHP 5 to run and obviously your server runs still PHP 4. Please upgrade to PHP 5 or ask you provider to change the settings for you. Nowadays PHP 5 is available virtually everywhere.

Thanks,
Bogdan

Your demo site have an erron in Doc Finder module:

Warning: include(assets/modules/docfinder/index.php) [function.include]: failed to open stream: No such file or directory in /srv/www/vhosts/trymodx.com/subdomains/docfinder/httpdocs/manager/processors/execute_module.processor.php(114) : eval()’d code on line 1

Warning: include(assets/modules/docfinder/index.php) [function.include]: failed to open stream: No such file or directory in /srv/www/vhosts/trymodx.com/subdomains/docfinder/httpdocs/manager/processors/execute_module.processor.php(114) : eval()’d code on line 1

Warning: include() [function.include]: Failed opening ’assets/modules/docfinder/index.php’ for inclusion (include_path=’/srv/www/vhosts/trymodx.com/subdomains/docfinder/httpdocs/manager/includes/’) in /srv/www/vhosts/trymodx.com/subdomains/docfinder/httpdocs/manager/processors/execute_module.processor.php(114) : eval()’d code on line 1

Thanks for the info, Doc Finder demo ist now running again.

Cheers,
Bogdan

Quote from: paulb at Dec 05, 2008, 04:33 PM

Also, on a separate topic, it would be helpful if you could replace all instances of <? (short php tag) with <?php (the long php tag), and all instances of <?= with <?php echo . Those of us who have the short tags turned off can’t use your module unless we search and replace all of those instances.

Thx for mention this Paul. It did bring me the strange behavior.
After made changes to the tags, it worked charm.
This is trully a great tool for the manager.

Here are the files for the php’s "short_open_tag = Off" version (like mine).
Only to save your time.

PS: I changed some hyperlinks to the images/icons folder.
Cheers.

Thanks for your contribution goldsky! I just merged your files to Doc Finder 1.5.2 and submitted it to the new repository which is now called "extras". Lets see, I hope the new version will be soon confirmed by an administrator and is available to everyone:

http://modxcms.com/extras.html?view=package/view&repository=10&package=103

Cheers,
Bogdan

I got error on the calendar thing.
Now I’m using the current (long tag and php’s minor changes) + modified vlacal include files (attached).
Just replace the files in \assets\modules\docfinder\js\vlaCal.v2\inc