Check Permissions v1.3 for osCmax

[tnier]

...then, a better explanation as to what the issue is, may help.

heres what my tools.php says

<?php
/*
$Id: tools.php 3 2006-05-27 04:59:07Z user $

osCMax Power E-Commerce
osCommerce Documentation by OSCdox :: osCommerce and osCMax documentation

Copyright 2006 osCMax2005 osCMax, 2002 osCommerce

Released under the GNU General Public License
*/
?>
<!-- tools //-->
<tr>
<td>
<?php
$heading = array();
$contents = array();

$heading[] = array('text' => BOX_HEADING_TOOLS,
'link' => tep_href_link(FILENAME_BACKUP, 'selected_box=tools'));

if ($selected_box == 'tools') {
// BOF: MOD - Admin w/access levels
// $contents[] = array('text' => '<a href="' . tep_href_link(FILENAME_BACKUP) . '" class="menuBoxContentLink">' . BOX_TOOLS_BACKUP . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_BANNER_MANAGER) . '" class="menuBoxContentLink">' . BOX_TOOLS_BANNER_MANAGER . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_CACHE) . '" class="menuBoxContentLink">' . BOX_TOOLS_CACHE . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_DEFINE_LANGUAGE) . '" class="menuBoxContentLink">' . BOX_TOOLS_DEFINE_LANGUAGE . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_FILE_MANAGER) . '" class="menuBoxContentLink">' . BOX_TOOLS_FILE_MANAGER . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_MAIL) . '" class="menuBoxContentLink">' . BOX_TOOLS_MAIL . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_NEWSLETTERS) . '" class="menuBoxContentLink">' . BOX_TOOLS_NEWSLETTER_MANAGER . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_SERVER_INFO) . '" class="menuBoxContentLink">' . BOX_TOOLS_SERVER_INFO . '</a><br>' .
// '<a href="' . tep_href_link(FILENAME_WHOS_ONLINE) . '" class="menuBoxContentLink">' . BOX_TOOLS_WHOS_ONLINE . '</a>');
$contents[] = array('text' => tep_admin_files_boxes(FILENAME_BACKUP, BOX_TOOLS_BACKUP, TOP) .
tep_admin_files_boxes(FILENAME_RECOVER_CART_SALES, BOX_TOOLS_RECOVER_CART, TOP) .
tep_admin_files_boxes(FILENAME_BANNER_MANAGER, BOX_TOOLS_BANNER_MANAGER, TOP) .
tep_admin_files_boxes(FILENAME_BATCH_PRINT, BOX_TOOLS_BATCH_CENTER, TOP) .
tep_admin_files_boxes(FILENAME_MAIL, BOX_TOOLS_MAIL, TOP) .
tep_admin_files_boxes(FILENAME_NEWSLETTERS, BOX_TOOLS_NEWSLETTER_MANAGER, TOP) .
tep_admin_files_boxes(FILENAME_PACKAGING, BOX_TOOLS_PACKAGING, TOP) .
tep_admin_files_boxes(FILENAME_UPS_BOXES_USED, BOX_TOOLS_UPS_BOXES_USED, TOP) .
tep_admin_files_boxes(FILENAME_QTPRODOCTOR, BOX_TOOLS_QTPRODOCTOR, TOP) .
tep_admin_files_boxes(FILENAME_CACHE, BOX_TOOLS_CACHE, TOP) .
tep_admin_files_boxes(FILENAME_SERVER_INFO, BOX_TOOLS_SERVER_INFO, TOP) .
tep_admin_files_boxes(FILENAME_WHOS_ONLINE, BOX_TOOLS_WHOS_ONLINE, TOP));
// EOF: MOD - Admin w/access levels
}

$box = new box;
echo $box->menuBox($heading, $contents);
?>
</td>
</tr>
<!-- tools_eof //-->


The directions say

3. Add this line to admin/includes/boxes/tools.php - [edit pgm]

tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS, BOX_TOOLS_CHECK_PERMISSIONS) .

after

tep_admin_files_boxes(FILENAME_FILE_MANAGER, BOX_TOOLS_FILE_MANAGER) .

there is no
tep_admin_files_boxes(FILENAME_FILE_MANAGER, BOX_TOOLS_FILE_MANAGER) .
to put
tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS, BOX_TOOLS_CHECK_PERMISSIONS) .
after

when i put both
tep_admin_files_boxes(FILENAME_FILE_MANAGER, BOX_TOOLS_FILE_MANAGER) .
tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS, BOX_TOOLS_CHECK_PERMISSIONS) .
in the tools.php
it says cant find file_manager.php

[blackhawk]

hey peeps its really not that bad - all that line a code is trying to do is pull the information into a box function (i.e. the box lines you see on your information.php page. its really not rocket science.

[pgmarshall]

Hi there,

The filemanager code was removed from osCMax v.2.0.4 due to security issues!

Therefore this line was removed

tep_admin_files_boxes(FILENAME_FILE_MANAGER, BOX_TOOLS_FILE_MANAGER) .

DO NOT REPLACE IT!

Therefore the line should read ...

put

tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS,   BOX_TOOLS_CHECK_PERMISSIONS) .

after one of the other lines ... for example ...

tep_admin_files_boxes(FILENAME_BATCH_PRINT, BOX_TOOLS_BATCH_CENTER, TOP)  .

But you can put it anywhere in the string depending where you want it to appear in the menu.

Hope this clears this up for you.

Regards,

[onlyme]

Having struggled with this problem myself, here is what I hope is a a clear solution for others like me, who have no prior experience of PHP coding etc.

The instructions say to do this:

********************

3. Add this line to admin/includes/boxes/tools.php - [edit pgm]

tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS, BOX_TOOLS_CHECK_PERMISSIONS) .

after

tep_admin_files_boxes(FILENAME_FILE_MANAGER, BOX_TOOLS_FILE_MANAGER) .

********************

This line: tep_admin_files_boxes(FILENAME_FILE_MANAGER, BOX_TOOLS_FILE_MANAGER) does not exist and that is OK. You should not add this line in because it is not supposed to be there. The instructions could do with updating, that's all.

You should continue with adding in the line: tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS, BOX_TOOLS_CHECK_PERMISSIONS) .

You must make sure that you include the full stop at the end of the line. What you also must do is ensure you add the line before this one: tep_admin_files_boxes(FILENAME_WHOS_ONLINE, BOX_TOOLS_WHOS_ONLINE, TOP));

You will notice that the line shown above ends with a semi-colon, whereas the lines of code above it all end with a full stop. I hadn't noticed this and had added the new line right at the end, after the one with the semi-colon and I couldn't get anything to work. As soon as I noticed that the final line ended differently to those above it and I relocated the new line, it all worked.

[ridexbuilder]

Good of you to spell things out in such a hand-holding manner but reads like something for a young teenagers school lesson, IMO.

If I was being real nasty:

No prior experience of PHP programming is required. Only a capability of using the grey matter between ones ears, to compare existing lines in the file. If you look at the list of items does it not correspond to what you see in the Admin menus? Is it a real leap of thought to see the similarity?

For the anally retentive let's get really down to the nitty gritty...

 tep_admin_files_boxes(FILENAME_CHECK_PERMISSIONS, BOX_TOOLS_CHECK_PERMISSIONS) . 
  

tep_admin_files_boxes - a function to add boxes (menu item in Tools) to the admin console.
FILENAME_CHECK_PERMISSIONS - use the file that checks permission....
etc.
The reason for the Caps. is that these are variables, to account for different languages and/or changes to filenames.

'Developers'/'Webmasters' (what a wide bunch of skill sets that covers!) might want to try out adding contributions. This implies having a modicum of experience in coding matters.
Shop owners should stick with 'officially' released packages.
Things would be so much easier.
As Met00's sig. shows...

[onlyme]

Sorry, I thought a forum was for users having difficulties who might benefit from straightforward responses especially if, like me, they have never touched PHP files before. I only joined this forum because I was trying to help, so it's a shame I've been met with such unnecessary sarcasm.

[ridexbuilder]

Easily upset, then?
You didn't notice the hints of tongue-in-cheek (in addition to the little dots at the end of menu items, apart from the last one ).