search

GM-Script-Writer-62850 / PHP-Scanner-Server

Allows you to use your Linux install as a web based scanner server thus allowing you to scan with any web enabled device; Now supports server side printing! (As of 1.4.11)
59 stars 17 forks source link

$CANDIR missing #10

Closed andi-blafasl closed 11 years ago

andi-blafasl commented 11 years ago

While searching through your code I have seen you implemented a $CANDIR variable in the 1.3 release. But you are note using it as a replacement for every /tmp/ part.

In line 748 you check for an existing file in /tmp/ but in line 861 you create the file in the new $CANDIR

Maybe you should set the $CANDIR somwhere at top of your index.php in your Variables section and replace the /tmp/ down in the "scan image" section.

Kind Regards, Andreas

GM-Script-Writer-62850 commented 11 years ago

Ok, i found it will have it fixed in a a few min

GM-Script-Writer-62850 commented 11 years ago

Fixed, will push it later when i fix some more stuff, thanks for finding these issues

GM-Script-Writer-62850 commented 11 years ago

This should be fixed now, clone git and test it, be sure to clear your browser's cache and rescan for scanners edit: http://www.mediafire.com/download/bjtd14l3v2wufji/PHP-Server-Scanner-1.3-1-dev.tar.bz2 - dev copy

GM-Script-Writer-62850 commented 11 years ago

@andi-blafasl I am going to go a bit off topic, could you go bug hunting in the current git version? https://github.com/GM-Script-Writer-62850/PHP-Scanner-Server/wiki/Change-Log i also make a update script, it is on the download page in the wiki

contrary to popular belief i really don't scan or print anything, this project is the main/only reason i use my scanner

andi-blafasl commented 11 years ago

I just checked out the latest git-version and tried it on my pi. Scanner worked out of the Box, no more workaround for invalid --source option needed. For the one Bug I found, I open a new issue.