Bugfix: External link bug

If you install concrete5 to document root and put page list block under
2nd level then the external link doesn't work.
add fileset file addition and removal events

added two events:
on_file_added_to_set($file_id, $file_set_id)
on_file_removed_from_set($file_id, $file_set_id)

it seems when using the UI and updating the sets a file is in
all associations are removed and then re-added based on the checkboxes
which causes the on_file_added_to_set event to be triggered multiple
times for sets the file is already in. However, when called
programatically these events will be fired as expected
Refactor getBlocks, a cleaner pull

Single clean re-work of
https://github.com/concrete5/concrete5/pull/842
Implementing t2
update jsmin to 1.1.2
display all pages by default in pagelist block

There's an internal limit of 20 by default. If we don't call
setItemsPerPage, we only get 20 pages but this isn't clear in the
block dialog. I simply removed the check and call setItemsPerPage(0)
if no value has been entered.
We could also display the number 20 when adding a new block, either
way is okay I think.
using anonymous functions to extend events

I was never very happy with creating models to extend events.
Sometimes I thought it would be nicer if we could simply pass a
function and guess what? We already can but we need to specify
some parameters not needed.
I've made the optional.
Allow return to home from error page

DIR_REL is often empty, and so the href attribute will be empty.
Using adding BASE_URL correctly allows returning to homepage.
return to home from greek yogurt blog

same as mentioned here: https://github.com/concrete5/concrete5/pull/884
Update web/concrete/core/jobs/remove_old_page_versions.php

All pages have been processed (not processes)
Add the chance to change the en_us language vars in po files

Add a new constant ENABLE_TRANSLATE_LOCALE_EN_US - default: false
Add Consitant Collection::getByHandle return value

* Collection::getByHandle() returns false if CacheLocal returns -1
* Collection::getByHandle() returns -1 if CacheLocal returns falsey
* Adds check before final return $ak to prevent returning -1 and instead
  return false

Main issue found with -1 return value is the Package::install() idiom
of `if(!CollectionAttributeKey::getByHandle('my_handle')) {` as
`!-1 === false`. This causes the installer to skip the installation of the
CollectionAttributeKey as it assumes it already installed.
Fixed datetime default mask when localized datetime is used.
Merge branch 'master' of https://github.com/concrete5/concrete5 into localized_datetime_fix
Made the fix make more sense.
space between selected keyword and attribute

There needs to be a space between the attribute and the word "selected".
It currently reads value="123"selected
Fix Bulk SEO Tool Listing and Handles

Fixes the Bulk SEO tool to list all entries
even if their attributes are not searchable
and accounts for caching changes by modifying
how the handle saving functions work.
 Prevent PHP54 default obj warn and return value

`$bt->controller = ` will throw 'Warning: Creating default object from
empty value' in PHP5.4+ if block handle is not found.

In PHP5.4- this will return a instance of StdObject not BlockType if
block handle is not found.

Can cause issue with `if(BlockType::getByHandle('foo')) {` package
installer idiom.

* Sets return value to `false` to remain consistant with -1 cache value
`return false` if block handle not found
* Eliminates uneeded `else` statements that follow `if` statements which
end in `return` statements
* Corrects typo in phpdoc comments
* Corrects phpdoc @return to mixed
Fix Firefox Version Action Buttons

Add 145px width for Firefox action buttons in version dialog.

Prevents wrap-around. Since buttons are actually pixel dependant, might
as well just admit it and add pixel defined space for it.
Change php PHP 5.2.4 From Option to Required

Changes installer to require PHP 5.2.4 as opposed to suggesting an
earlier verison.