PPL

Contact merge should generate a log entry

Bug #717175 reported by Christopher Adams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PPL
Fix Released
High
Bassel Safadi

Bug Description

This can be reported by contacts.php, ideally reporting the number of contacts merged.

Changed in ppl:
status: New → Confirmed
Changed in ppl:
milestone: none → 0.2
Bassel Safadi (bassel)
Changed in ppl:
assignee: nobody → Bassel Safadi (bassel)
importance: Undecided → High
Revision history for this message
Christopher Adams (christopheradams) wrote :

Use the following code in contacts.php to add log entries:

Variables:
    $current_date = time();
    $owner_id = $userid; //passed to the merge() method

If there are merges:
    $log_type = '1'; //Notice
    $message = "You merged your contacts." //Can this report the number of merges?

If there are no merges:
    $log_type = '3'; //Alert
    $message = "There are no new contacts to merge."

Here is the query:

    $db->query("INSERT INTO ppl_logs (owner_id, log_type, created_at, message) VALUES ('$owner_id', '$log_type', FROM_UNIXTIME($current_date), '$message')");

Bassel Safadi (bassel)
Changed in ppl:
status: Confirmed → In Progress
Bassel Safadi (bassel)
Changed in ppl:
status: In Progress → Fix Released
Revision history for this message
rejon (rejon) wrote : Re: [Bug 717175] Re: Contact merge should generate a log entry

did you test that shit? forealid! :)

On Sat, Feb 12, 2011 at 7:28 PM, Bassel Safadi <email address hidden> wrote:
> ** Changed in: ppl
>       Status: In Progress => Fix Released
>
> --
> You received this bug notification because you are a member of PPL
> Developers, which is subscribed to PPL.
> https://bugs.launchpad.net/bugs/717175
>
> Title:
>  Contact merge should generate a log entry
>

--
Jon Phillips
http://rejon.org/ | http://fabricatorz.com/
chat/skype: kidproto | irc: rejon
+1.415.830.3884 (global) | +1-510-499-0894 (sf)

Revision history for this message
Bassel Safadi (bassel) wrote :

ha, of course tested it

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.