Sensei
- Join Date:
- Jan 2004
- Posts:
- 63,513
- Plugin Contributions:
- 176
Ceon URI Mapping v4.x
Ryk:
by circumventing the zen_record_admin_activity function
huh? Did you actually mean "circumvent" as in "break/bypass/disable" for even the built-in features? or did you mean "we won't use [the] function in all our modules' own customised add/remove/update/edit activities"?
I strongly advise that you LEAVE EXISTING calls to zen_record_admin_activity ALONE.
You don't need to add more, but removing or disabling built-in functionality, especially things that aid in security, is not particularly prudent.
I'm aware of the content of your discussion with wilt. But he certainly didn't tell you to go ahead and cripple or remove built-in functionality.
I realize that may make distributing 1 set of files for all 1.5.x versions a little more challenging, but the bridge contributed by lat9 would simplify that concern easily.
.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole
Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.