Admin Profiles mod vs 139

Hi,

dont know if im being anal and annoying...

just about to try and install Admin Profiles on 139

in Mr K's install guide it says that:
"If you have installed or upgraded to Zen Cart 1.3.8 and have NOT made changes to admin\admin.php or admin\includes\languages\english.php, and have NOT since installation/upgrade applied any other add-ons that affect these files, then you may use the pre-patched files in the folder named "extras - patched files for ZC1.3.8"."

using win merge the file admin.php is the same between 138a and 139.

Missing the:

if (!defined('IS_ADMIN_FLAG')) {
die('Illegal Access');
}

at the begining of the file suggested in http://www.zen-cart.com/forum/showthread.php?t=102802

Is it not needed anymore?

You don't need that for admin.php. Never did.

You should use it for english.php. But you shouldn't use the admin profiles supplied version anyway with 1.3.9, as there have been some updates to the core file.

i see, i had noticed that /english.php had changed.

Will you be revamping it for 139?

Probably, but I would expect that AP to work as is provided the instructions are followed, so a higher priority for me will be those mods that won't and therefore need to be upgraded for 1.3.9.

Just played.. works great as far as i can tell without any problems...

As winmerge shows Admin/admin.php as the same between 138a and 139 i just used your patched version in your download.. but followed your instructions for 'Patch Three' for admin/includes/languages/english.php as this is a chamged file

Cheers,, will add to my list