Ett 1155 clean up

[liseli]

This PR consists of several features that have been removed from the Catalog. After removing the features, I updated the Smarty template to ensure variables are not empty.

I have commented out all the logic related to the $user variable. Although we know the $user is always Null in Holdings.php, a global $user is defined. I want to confirm this part with @moseshll to ensure it is safe to remove.

If $recordcount is not defined, I have set it to 0 by default in the template. However, this variable is initialized in services/Search/Home.php and in services/Record/Record.php

$recordCount =  $result['RecordCount'];

I would like to check if we should set up a default value on the PHP side

Additional things we should check:

• How to remove this error "NOTICE: PHP message: can't write resultIDs cookie, length is 14200"? --> Check the comments here
• The RecordUtils.php EZproxy logic that depends on $inst will never select the Flint proxy because inst is never set.

[aelkiss]

The result cookie & the ezproxy thing was handled here: #103

It looks like now there are some conflicts with this branch, just let me know if you'd like me to try to address those.

[liseli]

sys/VFUser.php can be removed too. We will have to remove the dependency from require_once 'sys/VFUser.php'; Search/Home.php, SearchExport.php and SearchExtructure.php

[moseshll]

This is on dev-2

[moseshll]

I think this is a good place to leave things.