admin
Karma: 57
|
Re:Some Comments - 2008/04/11 15:45
Good points, and thanks for your kind comments.
1. Agreed - I've changed the code ready for the next release.
2. This can be quite easily achieved by making a small alternation to c-classes/remository_select_Controller.php. Alter the code starting from around line 19 so that it reads:
| Code: |
function select($func) {
$interface =& remositoryInterface::getInstance();
require_once ($this->admin->v_classes_path.'remositoryFileListHTML.php');
if ('select' == $func AND $this->idparm) {
$container = $this->createContainer ();
if (!$this->repository->See_Containers_no_download AND !$container->isDownloadable($this->remUser)) {
echo "<span class='remositorymessage'>"._DOWN_RESTRICTED_WARN.'</span>';
return;
}
}
else $container =& new remositoryContainer();
if ('select' == $func) $subfolders = $container->getVisibleChildren($this->remUser);
else $subfolders = array();
if ('select' == $func AND $this->idparm == 0 AND count($subfolders) == 0) {
echo "<span class='remositorymessage'>"._DOWN_NO_VISITOR_CATS.'</span>';
return;
}
if (!$this->idparm AND 1 == count($subfolders)) {
$container = $subfolders[0];
$subfolders = $container->getVisibleChildren($this->remUser);
}
if ($container->windowtitle) $interface->SetPageTitle($container->windowtitle);
elseif ($container->name) $interface->SetPageTitle($container->name);
$page = remositoryRepository::getParam($_REQUEST, 'page', 1);
|
and Remository will go straight into the top level container if it is the only one.
3-5. You're quite right that extra flexibility would be good - my only real concern is that the configuration interface gets too complicated 
Martin Brampton aka Counterpoint
http://black-sheep-research.com
http://mamboguru.com |
Forum
