Fixes #4422 pulling in Ismayil's changes so that we use the module view for settings

6 files changed, 75 insertions, 130 deletions

diff --git a/views/default/core/settings/account/default_access.php b/views/default/core/settings/account/default_access.php
index e33211460..690f2714a 100644
--- a/views/default/core/settings/account/default_access.php
+++ b/views/default/core/settings/account/default_access.php
@@ -12,22 +12,14 @@ if (elgg_get_config('allow_user_default_access')) {
 		if (false === ($default_access = $user->getPrivateSetting('elgg_default_access'))) {
 			$default_access = elgg_get_config('default_access');
 		}
-?>
-<div class="elgg-module elgg-module-info">
-	<div class="elgg-head">
-		<h3><?php echo elgg_echo('default_access:settings'); ?></h3>
-	</div>
-	<div class="elgg-body">
-		<p>
-		<?php echo elgg_echo('default_access:label'); ?>:
-		<?php
 
-			echo elgg_view('input/access', array('name' => 'default_access', 'value' => $default_access));
+		$title = elgg_echo('default_access:settings');
+		$content = elgg_echo('default_access:label') . ': ';
+		$content .= elgg_view('input/access', array(
+			'name' => 'default_access',
+			'value' => $default_access,
+		));
 
-		?>
-		</p>
-	</div>
-</div>
-<?php
+		echo elgg_view_module('info', $title, $content);
 	}
-}
\ No newline at end of file
+}
diff --git a/views/default/core/settings/account/email.php b/views/default/core/settings/account/email.php
index fda83e795..4bcdbb100 100644
--- a/views/default/core/settings/account/email.php
+++ b/views/default/core/settings/account/email.php
@@ -9,19 +9,11 @@
 $user = elgg_get_page_owner_entity();
 
 if ($user) {
-?>
-<div class="elgg-module elgg-module-info">
-	<div class="elgg-head">
-		<h3><?php echo elgg_echo('email:settings'); ?></h3>
-	</div>
-	<div class="elgg-body">
-		<p>
-			<?php echo elgg_echo('email:address:label'); ?>:
-			<?php
-			echo elgg_view('input/email',array('name' => 'email', 'value' => $user->email));
-			?>
-		</p>
-	</div>
-</div>
-<?php
-}
\ No newline at end of file
+	$title = elgg_echo('email:settings');
+	$content = elgg_echo('email:address:label') . ': ';
+	$content .= elgg_view('input/email', array(
+		'name' => 'email',
+		'value' => $user->email,
+	));
+	echo elgg_view_module('info', $title, $content);
+}
diff --git a/views/default/core/settings/account/language.php b/views/default/core/settings/account/language.php
index 3a992966f..b36057422 100644
--- a/views/default/core/settings/account/language.php
+++ b/views/default/core/settings/account/language.php
@@ -9,23 +9,12 @@
 $user = elgg_get_page_owner_entity();
 
 if ($user) {
-?>
-<div class="elgg-module elgg-module-info">
-	<div class="elgg-head">
-		<h3><?php echo elgg_echo('user:set:language'); ?></h3>
-	</div>
-	<div class="elgg-body">
-		<p>
-			<?php echo elgg_echo('user:language:label'); ?>:
-			<?php
-			echo elgg_view("input/dropdown", array(
-				'name' => 'language',
-				'value' => $user->language,
-				'options_values' => get_installed_translations()
-			));
-			?>
-		</p>
-	</div>
-</div>
-<?php
-}
\ No newline at end of file
+	$title = elgg_echo('user:set:language');
+	$content = elgg_echo('user:language:label') . ': ';
+	$content .= elgg_view("input/dropdown", array(
+		'name' => 'language',
+		'value' => $user->language,
+		'options_values' => get_installed_translations()
+	));
+	echo elgg_view_module('info', $title, $content);
+}
diff --git a/views/default/core/settings/account/name.php b/views/default/core/settings/account/name.php
index f719a84de..e356146a3 100644
--- a/views/default/core/settings/account/name.php
+++ b/views/default/core/settings/account/name.php
@@ -7,20 +7,15 @@
  */
 
 $user = elgg_get_page_owner_entity();
-?>
-<div class="elgg-module elgg-module-info">
-  <div class="elgg-head">
-		<h3><?php echo elgg_echo('user:name:label'); ?></h3>
-	</div>
-	<div class="elgg-body">
-		<p>
-			<?php echo elgg_echo('name'); ?>:
-			<?php
-			echo elgg_view('input/text', array('name' => 'name', 'value' => $user->name));
-			?>
-		</p>
-	</div>
-</div>
-<?php
-// need the user's guid to make sure the correct user gets updated
-echo elgg_view('input/hidden', array('name' => 'guid', 'value' => $user->guid));
+if ($user) {
+	$title = elgg_echo('user:name:label');
+	$content = elgg_echo('name') . ': ';
+	$content .= elgg_view('input/text', array(
+		'name' => 'name',
+		'value' => $user->name,
+	));
+	echo elgg_view_module('info', $title, $content);
+
+	// need the user's guid to make sure the correct user gets updated
+	echo elgg_view('input/hidden', array('name' => 'guid', 'value' => $user->guid));
+}
diff --git a/views/default/core/settings/account/notifications.php b/views/default/core/settings/account/notifications.php
index 51292b2c9..c212b3886 100644
--- a/views/default/core/settings/account/notifications.php
+++ b/views/default/core/settings/account/notifications.php
@@ -9,33 +9,20 @@
 global $NOTIFICATION_HANDLERS;
 $notification_settings = get_user_notification_settings(elgg_get_page_owner_guid());
 
-?>
-<div class="elgg-module elgg-module-info">
-	<div class="elgg-head">
-		<h3><?php echo elgg_echo('notifications:usersettings'); ?></h3>
-	</div>
-	<div class="elgg-body">
+$title = elgg_echo('notifications:usersettings');
 
-<p><?php echo elgg_echo('notifications:methods'); ?>
+$rows = '';
 
-<table>
-<?php
-	// Loop through options
-	foreach ($NOTIFICATION_HANDLERS as $k => $v) {
-?>
-		<tr>
-			<td><?php echo elgg_echo("notification:method:$k"); ?>: </td>
-
-			<td>
-<?php
+// Loop through options
+foreach ($NOTIFICATION_HANDLERS as $k => $v) {
 
 	if ($notification_settings->$k) {
 		$val = "yes";
 	} else {
 		$val = "no";
 	}
-	
-	echo elgg_view('input/radio', array(
+
+	$radio = elgg_view('input/radio', array(
 		'name' => "method[$k]",
 		'value' => $val,
 		'options' => array(
@@ -44,12 +31,14 @@ $notification_settings = get_user_notification_settings(elgg_get_page_owner_guid
 		),
 	));
 
-?>
-			</td>
-		</tr>
-<?php
-	}
-?>
-</table>
-	</div>
-</div>
\ No newline at end of file
+	$cells = '<td class="prm pbl">' . elgg_echo("notification:method:$k") . ': </td>';
+	$cells .= "<td>$radio</td>";
+
+	$rows .= "<tr>$cells</tr>";
+}
+
+
+$content = elgg_echo('notifications:methods');
+$content .= "<table>$rows</table>";
+
+echo elgg_view_module('info', $title, $content);
diff --git a/views/default/core/settings/account/password.php b/views/default/core/settings/account/password.php
index a80f3df6b..4857034b2 100644
--- a/views/default/core/settings/account/password.php
+++ b/views/default/core/settings/account/password.php
@@ -1,45 +1,33 @@
 <?php
+
 /**
  * Provide a way of setting your password
  *
  * @package Elgg
  * @subpackage Core
  */
-
 $user = elgg_get_page_owner_entity();
 
 if ($user) {
-?>
-<div class="elgg-module elgg-module-info">
-	<div class="elgg-head">
-		<h3><?php echo elgg_echo('user:set:password'); ?></h3>
-	</div>
-	<div class="elgg-body">
-		<?php
-			// only make the admin user enter current password for changing his own password.
-			if (!elgg_is_admin_logged_in() || elgg_is_admin_logged_in() && $user->guid == elgg_get_logged_in_user_guid()) {
-		?>
-		<p>
-		<?php echo elgg_echo('user:current_password:label'); ?>:
-		<?php
-			echo elgg_view('input/password', array('name' => 'current_password'));
-		?>
-		</p>
-		<?php } ?>
+	$title = elgg_echo('user:set:password');
 
-		<p>
-		<?php echo elgg_echo('user:password:label'); ?>:
-		<?php
-			echo elgg_view('input/password', array('name' => 'password'));
-		?>
-		</p>
+	// only make the admin user enter current password for changing his own password.
+	$admin = '';
+	if (!elgg_is_admin_logged_in() || elgg_is_admin_logged_in() && $user->guid == elgg_get_logged_in_user_guid()) {
+		$admin .= elgg_echo('user:current_password:label') . ': ';
+		$admin .= elgg_view('input/password', array('name' => 'current_password'));
+		$admin = "<p>$admin</p>";
+	}
 
-		<p>
-		<?php echo elgg_echo('user:password2:label'); ?>: <?php
-			echo elgg_view('input/password', array('name' => 'password2'));
-		?>
-		</p>
-	</div>
-</div>
-<?php
-}
\ No newline at end of file
+	$password = elgg_echo('user:password:label') . ': ';
+	$password .= elgg_view('input/password', array('name' => 'password'));
+	$password = "<p>$password</p>";
+
+	$password2 = elgg_echo('user:password2:label') . ': ';
+	$password2 .= elgg_view('input/password', array('name' => 'password2'));
+	$password2 = "<p>$password2</p>";
+
+	$content = $admin . $password . $password2;
+
+	echo elgg_view_module('info', $title, $content);
+}