Completed
Pull Request — master (#165)
by Stephanie
02:17
created
classes/models/FrmInbox.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -115,7 +115,6 @@
 block discarded – undo
115 115
 	}
116 116
 
117 117
 	/**
118
-	 * @param string $timestamp in time format.
119 118
 	 */
120 119
 	public function mark_read( $key ) {
121 120
 		$this->set_messages();
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 			$read    = isset( $message['read'] ) && ! empty( $message['read'] );
80 80
 			$expired = isset( $message['expires'] ) && ! empty( $message['expires'] ) && $message['expires'] < time();
81 81
 			if ( $read && $expired ) {
82
-				unset( $messages[ $t ] );
82
+				unset( $messages[$t] );
83 83
 				$removed = true;
84 84
 			}
85 85
 		}
@@ -97,12 +97,12 @@  discard block
 block discarded – undo
97 97
 		$this->set_messages( 'skip' );
98 98
 		$time = isset( $message['time'] ) ? $message['time'] : time();
99 99
 
100
-		if ( isset( $this->messages[ $message['key'] ] ) && ! isset( $message['force'] ) ) {
100
+		if ( isset( $this->messages[$message['key']] ) && ! isset( $message['force'] ) ) {
101 101
 			// Don't replace messages unless required.
102 102
 			return;
103 103
 		}
104 104
 
105
-		$this->messages[ $message['key'] ] = array(
105
+		$this->messages[$message['key']] = array(
106 106
 			'created' => $time,
107 107
 			'message' => $message['message'],
108 108
 			'subject' => $message['subject'],
@@ -119,14 +119,14 @@  discard block
 block discarded – undo
119 119
 	 */
120 120
 	public function mark_read( $key ) {
121 121
 		$this->set_messages();
122
-		if ( ! isset( $this->messages[ $key ] ) ) {
122
+		if ( ! isset( $this->messages[$key] ) ) {
123 123
 			return;
124 124
 		}
125 125
 
126
-		if ( ! isset( $this->messages[ $key ]['read'] ) ) {
127
-			$this->messages[ $key ]['read'] = array();
126
+		if ( ! isset( $this->messages[$key]['read'] ) ) {
127
+			$this->messages[$key]['read'] = array();
128 128
 		}
129
-		$this->messages[ $key ]['read'][ get_current_user_id() ] = time();
129
+		$this->messages[$key]['read'][get_current_user_id()] = time();
130 130
 
131 131
 		$this->update_list();
132 132
 	}
@@ -135,8 +135,8 @@  discard block
 block discarded – undo
135 135
 		$messages = $this->get_messages();
136 136
 		$user_id  = get_current_user_id();
137 137
 		foreach ( $messages as $t => $message ) {
138
-			if ( isset( $message['read'] ) && isset( $message['read'][ $user_id ] ) ) {
139
-				unset( $messages[ $t ] );
138
+			if ( isset( $message['read'] ) && isset( $message['read'][$user_id] ) ) {
139
+				unset( $messages[$t] );
140 140
 			}
141 141
 		}
142 142
 		return $messages;
Please login to merge, or discard this patch.
classes/views/frm-settings/inbox.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
 			</h3>
15 15
 			<span class="frm_inbox_date">
16 16
 				<?php
17
-				if ( ! isset( $message['read'] ) || ! isset( $message['read'][ get_current_user_id() ] ) ) {
17
+				if ( ! isset( $message['read'] ) || ! isset( $message['read'][get_current_user_id()] ) ) {
18 18
 					$inbox->mark_read( $key );
19 19
 					?>
20 20
 					<span class="frm_inbox_unread"></span>
Please login to merge, or discard this patch.
classes/controllers/FrmAppController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
 	private static function get_notice_count() {
25 25
 		FrmFormMigratorsHelper::maybe_add_to_inbox();
26 26
 
27
-		$inbox  = new FrmInbox();
27
+		$inbox = new FrmInbox();
28 28
 		return $inbox->unread_html();
29 29
 	}
30 30
 
Please login to merge, or discard this patch.