Fixes #4982 adds confirm dialog to profile field reset

diff --git a/CHANGES.txt b/CHANGES.txt
index acfb7055ef81c61925c7eedc45d7c8b0ab87c495..8687b3d2a0b054730dd0423bbb07c5f8b25bee03 100644 (file)
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,3 +1,15 @@
+Version 1.8.13
+(January xx, 2013 from https://github.com/Elgg/Elgg/tree/1.8)
+  Contributing Developers:
+   * Cash Costello
+   * Cristo Rabani
+
+  Security Fixes:
+
+  Enhancements:
+       * Added confirm dialog for resetting profile fields (adds language string profile:resetdefault:confirm)
+
+
 Version 1.8.12
 (January 4th, 2013 from https://github.com/Elgg/Elgg/tree/1.8)
   Contributing Developers:
@@ -7,7 +19,6 @@ Version 1.8.12
    * Matt Beckett
    * Paweł Sroka
    * Sem
-   * Srokap
    * Steve Clay
 
   Bugfixes:
@@ -34,6 +45,7 @@ Version 1.8.12
    * Twitter: Login with twitter supports persistent login and correctly forwards
      after login.
 
+
 Version 1.8.11
 (December 5th, 2012 from https://github.com/Elgg/Elgg/tree/1.8)
 

diff --git a/languages/en.php b/languages/en.php
index f3acc50ee083705e8518c1b094441cc790e8037e..00a825b4d47c93a574c938c8777d4d6b90155da3 100644 (file)
--- a/languages/en.php
+++ b/languages/en.php
@@ -405,7 +405,8 @@ $english = array(
        'profile:editdefault:delete:fail' => 'Removed default profile item field failed',
        'profile:editdefault:delete:success' => 'Profile field deleted',
        'profile:defaultprofile:reset' => 'Profile fields reset to the system default',
-       'profile:resetdefault' => 'Reset default profile',
+       'profile:resetdefault' => 'Reset profile fields to system defaults',
+       'profile:resetdefault:confirm' => 'Are you sure you want to delete your custom profile fields?',
        'profile:explainchangefields' => "You can replace the existing profile fields with your own using the form below. \n\n Give the new profile field a label, for example, 'Favorite team', then select the field type (eg. text, url, tags), and click the 'Add' button. To re-order the fields drag on the handle next to the field label. To edit a field label - click on the label's text to make it editable. \n\n At any time you can revert back to the default profile set up, but you will lose any information already entered into custom fields on profile pages.",
        'profile:editdefault:success' => 'New profile field added',
        'profile:editdefault:fail' => 'Default profile could not be saved',

diff --git a/views/default/admin/appearance/profile_fields.php b/views/default/admin/appearance/profile_fields.php
index 91d14b874991afd7a4195fda0fe955a73bf54d9d..f1d78c19fa5fa5856204709df66d0256eeeaaab2 100644 (file)
--- a/views/default/admin/appearance/profile_fields.php
+++ b/views/default/admin/appearance/profile_fields.php
@@ -5,12 +5,20 @@
 
 $add = elgg_view_form('profile/fields/add', array('class' => 'elgg-form-settings'), array());
 $list = elgg_view('admin/appearance/profile_fields/list');
-$reset = elgg_view_form('profile/fields/reset', array(), array());
+
+$reset = elgg_view('output/confirmlink', array(
+       'text' => elgg_echo('reset'),
+       'href' => 'action/profile/fields/reset',
+       'title' => elgg_echo('profile:resetdefault'),
+       'confirm' => elgg_echo('profile:resetdefault:confirm'),
+       'class' => 'elgg-button elgg-button-cancel',
+       'is_trusted' => 'true',
+));
 
 $body = <<<__HTML
 $add
 $list
-$reset
+<div class="mtl">$reset</div>
 __HTML;
 
 echo $body;

diff --git a/views/default/forms/profile/fields/reset.php b/views/default/forms/profile/fields/reset.php
deleted file mode 100644 (file)
index c0bb1b7..0000000
--- a/views/default/forms/profile/fields/reset.php
+++ /dev/null
@@ -1,12 +0,0 @@
-<?php
-/**
- * Reset profile fields form
- */
-
-echo '<div class="elgg-foot">';
-$params = array(
-       'value' => elgg_echo('profile:resetdefault'),
-       'class' => 'elgg-button-cancel',
-);
-echo elgg_view('input/submit', $params);
-echo '</div>';