Fixing the agreement icon and WPCS updates

Author: krugazul

includes/classes/class-email-invoice.php

@@ -57,8 +57,8 @@ public function send_invoice( $form_id, $currency, $amount, $name, $email, $code
 		$this->email       = stripslashes( $email );
 		$this->referer_url = $referer_url;
 
-		/* Translators: %1$s is the currency code, %2$s is the formatted amount */
 		$this->subject = sprintf(
+			// Translators: %1$s is the currency code, %2$s is the formatted amount
 			__( 'Payment Invoice for %1$s %2$s', 'text-domain' ),
 			$currency,
 			number_format( $amount )

includes/classes/class-email-receipt.php

@@ -160,16 +160,16 @@ public function get_html_body() {
 															<p style="font-family:Helvetica,Arial,sans-serif;font-size:15px;line-height:23px;margin-top:8px;margin-bottom:16px">
 																<?php 
 																printf(
-																	/* translators: %s: Amount */
-																	esc_html__( 'Amount : %s %s', 'pff-paystack' ),
+																	// translators: %1$s is the currency code, %2$s is the formatted amount
+																	esc_html__( 'Amount : %1$s %2$s', 'pff-paystack' ),
 																	esc_html( $this->currency ),
 																	number_format_i18n( $this->amount )
 																);
 																?><br>
 																<?php 
 																printf(
-																	/* translators: %s: Email */
-																	esc_html__( 'Email : %s', 'pff-paystack' ),
+																	/* translators: %1$s is the Email */
+																	esc_html__( 'Email : %1$s', 'pff-paystack' ),
 																	esc_html( $this->email )
 																);
 																?><br>

includes/classes/class-field-shortcodes.php

@@ -57,7 +57,10 @@ public function text_field( $atts ) {
 
 		$code .= '</label>
 			<div class="input">
-				<input type="text" id="' . esc_attr( $id ) . '" name="' . esc_attr( $name ) . '" placeholder="' . sprintf( esc_attr__( 'Enter %s', 'pff-paystack' ), $name ) . '" ' . esc_attr( $required ) . ' /></div></div>';
+				<input type="text" id="' . esc_attr( $id ) . '" name="' . esc_attr( $name ) . '" placeholder="' . 
+				// translators: %s: input field name to be entered by the user
+				sprintf( esc_attr__( 'Enter %s', 'pff-paystack' ), $name ) . 
+				'" ' . esc_attr( $required ) . ' /></div></div>';
 
 		return $code;
 	}

includes/classes/class-form-shortcode.php

@@ -373,11 +373,11 @@ public function get_amount_field() {
 	 */
 	public function get_agreement_field() {
 		$html = '';
-		if ( $this->meta['useagreement'] == 'yes' ) {
+		if ( 'yes' === $this->meta['useagreement'] ) {
 		$html = '<div class="span12 unit">
 					<label class="checkbox">
 						<input type="checkbox" name="agreement" id="pf-agreement" required value="yes">
-						<i></i>
+						<i id="pf-agreementicon"></i>
 						Accept terms <a target="_blank" href="' . esc_url( $this->meta['agreementlink'] ) . '">Link</a>
 					</label>
 				</div><br>';

includes/classes/class-form-submit.php

@@ -215,6 +215,7 @@ public function process_images() {
 				if ( $value['size'] > 0 ) {
 					if ( $value['size'] > $max_file_size ) {
 						$response['result']  = 'failed';
+						// translators: %s: maximum upload file size in MB
 						$response['message'] = sprintf( __( 'Max upload size is %sMB', 'pff-paystack' ), $this->meta['filelimit'] );
 						exit( wp_json_encode( $response ) );
 					} else {