History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: OX-3311
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Lukasz Wikierski
Reporter: andrzej.swedrzynski
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
OpenX Ad Server

Audit logging missing from new campaign/zone linking page

Created: 18/Jun/08 10:14 AM   Updated: 22/Apr/09 08:07 AM
Component/s: None
Affects Version/s: None
Fix Version/s: Milestone 23, OpenX 2.7.22-dev
Security Level: Public (All users can see these issues)

Time Tracking:
Original Estimate: 6h
Original Estimate - 6h
Remaining Estimate: 2.92h
Time Spent - 3.08h Remaining Estimate - 2.92h
Time Spent: 3.08h
Time Spent - 3.08h Remaining Estimate - 2.92h

Issue Links:
Reference
 


 Description  « Hide
A new high-performance function which links zones to a campaign unfortunately uses raw SQL query, therefore is not logged into audit trail. We should look into how we can add audit-trail to this functionality. This is a follow-up work for OX-2890.

 All   Comments   Work Log   Change History   FishEye   Crucible   Builds      Sort Order: Ascending order - Click to sort in descending order
Lukasz Wikierski - 19/Jun/08 08:19 AM - edited
When changing methods linking zones to a campaign, don't forget about Oracle limitation ORA-01795: maximum number of expressions in a list is 1000

At first look there are 2 ways of adding audit log:

  1. we can use current query to get list of banner-zones to link (avoid checking if it is possible to link banner<->zone in PHP). Then in loop add all links by DB_Objects
    • pros: no changes in audit log, this should be faster than methods in Admin_DA
    • cons: could be slow (eg. 10 banner in campaign and 1000 zones gives 10000 inserts by DB_Objects)
  2. leave this query as is, but add new auditlog entry (eg Mass Ad Zone Association) with parameters list of zones and list of banners, or list of pairs banner-zone (more data but more accurate)
    • pros: we stay with current efficiency
    • cons: in future we'll not able to search Mass Ad Zone Association for given banner in auditlog (or it will be costly operation - unserialize all parameters from all "Mass Ad Zone Association" logs)



Lukasz Wikierski - 19/Sep/08 08:49 AM
Fixed:
  • DB_DataObjects used to insert and delete associations, so audit logs are done automatically

Checked the logic in linking zones/banners/campaigns pages. Found one error OX-4092 on zone-include.php (zone linking to campaigns/ banners) page


Sue Houghton - 23/Sep/08 04:11 PM
closed in 2.7.23-beta