picky, picky--some of this is for consistency. such as Dir vs Directory.. drupal vs Drupal. one is spelling.

I had noticed a reference to the INSTALL.txt in one of the files, but that file is empty(added the install steps).
The README is an update for robots.txt

You are a perfectionist, so I am just having a look..
WOW,, You can sure write some compound and complicated functions !! i am just starting to learn php..
// Get Grandkids .. lol, good way to label that.

That about sums it up.. I'm done for now..
Have a look, use what you want.

CommentFileSizeAuthor
#7 watchdog_boost.module.patch695 bytesdbeall
#2 update2_INSTALL.txt.patch4.04 KBdbeall
#1 words2_boost.admin_.inc_.patch5.43 KBdbeall
words_boost.module.patch2.22 KBdbeall
words_boost.admin_.inc_.patch5.43 KBdbeall
update_README.patch906 bytesdbeall
update_INSTALL.txt.patch3.8 KBdbeall

Comments

dbeall’s picture

dbeall commented
StatusFileSize
new words2_boost.admin_.inc_.patch5.43 KB

had one word wrong in this one.. should double check things..

dbeall’s picture

dbeall commented
StatusFileSize
new update2_INSTALL.txt.patch4.04 KB

PEKAC.. I must be getting tired.. forgot robots.txt part in the install text..

mikeytown2’s picture

mikeytown2 commented

committed all but the install.txt file.

I thought we where going to keep the install stuff in the readme file; or is it best to leave it out? I can remove the install.txt file, just have been lazy about it... what are your thoughts on this?

dbeall’s picture

dbeall commented

I have to look through the other files,, I think I saw someplace that referred to it which is why i did that.
here it is .module file. I suppose that could be changed to readme.txt, just never thought about it at the time.. I was busy code reading and admiring your work..

/**
* Returns the full directory path to the static file cache directory.
*
* @param $host
* Host name. Example: example.com
* @param $absolute
* Give path from system root if true. If false give path from web root.
*/
function boost_cache_directory($host = NULL, $absolute = TRUE) {
global $base_url;
if ($base_url == "http://") {
if (BOOST_VERBOSE >= 1) {
watchdog('boost', 'base_url is not set in your settings.php file. Please read #7 in boosts INSTALL.txt file.', array(), WATCHDOG_NOTICE);
}

dbeall’s picture

dbeall commented

it's not "#7" in readme.. it's in "IMPORTANT NOTES AND TROUBLESHOOTING"

dbeall’s picture

dbeall commented

lol, it's not in the install.txt anyway either...

dbeall’s picture

dbeall commented
StatusFileSize
new watchdog_boost.module.patch695 bytes

maybe this will do it

mikeytown2’s picture

mikeytown2 commented
Status: Needs review » Fixed

Sounds good. Committed!

Status: Fixed » Closed (fixed)

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