diff --git a/include/class-Log.php b/include/class-Log.php index 6639ac5..256ccee 100644 --- a/include/class-Log.php +++ b/include/class-Log.php @@ -1,221 +1,216 @@ . // make sure that this class is loaded at startup class_exists( User::class ); class_exists( Domain::class ); class_exists( Mailbox::class ); trait LogTrait { /** * Get the log actor name * * @return string */ public function getLogActorFirm() { return User::firm( $this->get( 'actor_uid' ) ); } /** * Get the action family * * @return string */ public function getLogFamily() { return $this->get( 'log_family' ); } /** * Get the action */ public function getLogAction() { return $this->get( 'log_action' ); } /** * Get the action */ public function getLogDate() { return $this->get( 'log_timestamp' ); } /** * Get the log message * * @param array $args Arguments * @return self */ public function getLogMessage( $args ) { $family = $this->getLogFamily(); $action = $this->getLogAction(); // trigger the right message family switch( $family ) { case 'domain': return self::domainMessage( $action, $this, $args ); case 'mailbox': return self::mailboxMessage( $action, $this, $args ); } return self::unknownAction( $family, $action ); } /** - * Get the log message alongside the date + * Get the log message alongside the date and the actor name * * @param array $args Arguments * @return self */ - public function getLogMessageWithDate( $args ) { + public function getLogMessageWithDateAndUser( $args ) { + + $actor = $args['actor'] ?? $this; + + // create the Actor firm from the passed User object or from the Log + $actor_firm = $actor instanceof User + ? $actor->getUserFirm() + : $actor->getLogActorFirm(); + return sprintf( - "%s - %s", + "%s - %s %s", $this->getLogDate()->format( __( "Y-m-d H:i" ) ), + $actor_firm, $this->getLogMessage( $args ) ); } protected function normalizeLog() { $this->datetimes( 'log_timestamp' ); } } /** * A generic log of an action * * Something happened. Dunno what. */ class Log extends Queried { use LogTrait; use UserTrait; use DomainTrait; use MailboxTrait; public function __construct() { $this->normalizeLog(); } /** * Database table name */ const T = 'log'; /** * Generate a Domain-related message * * @param string $action The related action name * @param object $log * @param array $args Arguments * @return string Message */ public static function domainMessage( $action, $log, $args ) { /** * You can pass some objects to build the message: * * A complete 'actor' User object * A complete 'domain' Domain object */ - $actor = $args['actor'] ?? $log; $domain = $args['domain'] ?? $log; $plan = $args['plan'] ?? $log; - // create the Actor firm from the passed User object or from the Log - $actor_firm = $actor instanceof User - ? $actor->getUserFirm() - : $actor->getLogActorFirm(); - switch( $action ) { // an administrator has changed the Plan for a Domain case 'plan.change': return sprintf( - __( "%s changed the Plan for %s to %s" ), - $actor_firm, + __( "changed the Plan for %s to %s" ), $domain->getDomainFirm(), esc_html( $plan->getPlanName() ) ); } // default dummy message return self::unknownAction( 'domain', $action ); } /** * Generate a Mailbox-related message * * @param string $action The related action name * @param object $log * @param array $args Arguments * @return string Message */ public static function mailboxMessage( $action, $log, $args ) { /** * You can pass some objects to build the message: * * A complete 'actor' User object * A complete 'domain' Domain object * A complete 'mailbox' Mailbox object */ $actor = $args['actor'] ?? $log; $domain = $args['domain'] ?? $log; $mailbox = $args['mailbox'] ?? $log; - // create the Actor firm from the passed User object or from the Log - $actor_firm = $actor instanceof User - ? $actor->getUserFirm() - : $actor->getLogActorFirm(); - $mailbox_firm = Mailbox::firm( $mailbox->getMailboxUsername(), $domain->getDomainName() ); // trigger the right action message switch( $action ) { // the mailbox was created case 'create': return sprintf( - __( "%s created the mailbox %s" ), - $actor_firm, + __( "created the mailbox %s" ), $mailbox_firm ); case 'description.change': return sprintf( - __( "%s edited description of %s" ), - $actor_firm, + __( "edited description of %s" ), $mailbox_firm ); } // default dummy message return self::unknownAction( 'mailbox', $action ); } private static function unknownAction( $family, $action ) { return esc_html( sprintf( __( "misterious action about %s (%s)" ), $family, $action ) ); } } diff --git a/template/activity-panel.php b/template/activity-panel.php index 26424e8..56e3050 100644 --- a/template/activity-panel.php +++ b/template/activity-panel.php @@ -1,36 +1,36 @@ . /* * This is the template for the activity panel * * Called from * include/class-ActivityPanel.php - the_link() function * * Available variables: * $message_args array Arguments * $query QueryLog The query log */ ?>