From 5ecd127960654865716742282e1c5db66ca398af Mon Sep 17 00:00:00 2001 From: cash Date: Sat, 23 Feb 2013 11:29:13 -0500 Subject: Fixes #4971 serving languages from ajax page handler rather than simplecache (which cannot handle languages) --- js/lib/languages.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) (limited to 'js') diff --git a/js/lib/languages.js b/js/lib/languages.js index 99a1ba0ee..44ea56d2b 100644 --- a/js/lib/languages.js +++ b/js/lib/languages.js @@ -28,13 +28,8 @@ elgg.reload_all_translations = function(language) { var lang = language || elgg.get_language(); var url, options; - if (elgg.config.simplecache_enabled) { - url = 'cache/js/default/languages/' + lang + '.' + elgg.config.lastcache + '.js'; - options = {}; - } else { - url = 'ajax/view/js/languages'; - options = {data: {language: lang}}; - } + url = 'ajax/view/js/languages'; + options = {data: {language: lang}}; options['success'] = function(json) { elgg.add_translation(lang, json); -- cgit v1.2.3 From 06c3b6e3c41c629e510c55199bd19914273b0e64 Mon Sep 17 00:00:00 2001 From: cash Date: Sat, 23 Feb 2013 14:16:29 -0500 Subject: Fixes #4997 stop requesting a token after a failed request --- engine/lib/actions.php | 14 ++++++++++++-- js/lib/security.js | 33 ++++++++++----------------------- languages/en.php | 4 ++-- 3 files changed, 24 insertions(+), 27 deletions(-) (limited to 'js') diff --git a/engine/lib/actions.php b/engine/lib/actions.php index 53b185dea..ac6325813 100644 --- a/engine/lib/actions.php +++ b/engine/lib/actions.php @@ -252,10 +252,20 @@ function validate_action_token($visibleerrors = TRUE, $token = NULL, $ts = NULL) register_error(elgg_echo('actiongatekeeper:pluginprevents')); } } else if ($visibleerrors) { - register_error(elgg_echo('actiongatekeeper:timeerror')); + // this is necessary because of #5133 + if (elgg_is_xhr()) { + register_error(elgg_echo('js:security:token_refresh_failed', array(elgg_get_site_url()))); + } else { + register_error(elgg_echo('actiongatekeeper:timeerror')); + } } } else if ($visibleerrors) { - register_error(elgg_echo('actiongatekeeper:tokeninvalid')); + // this is necessary because of #5133 + if (elgg_is_xhr()) { + register_error(elgg_echo('js:security:token_refresh_failed', array(elgg_get_site_url()))); + } else { + register_error(elgg_echo('actiongatekeeper:tokeninvalid')); + } } } else { if (! empty($_SERVER['CONTENT_LENGTH']) && empty($_POST)) { diff --git a/js/lib/security.js b/js/lib/security.js index 61aa1cfcd..af02824a6 100644 --- a/js/lib/security.js +++ b/js/lib/security.js @@ -7,6 +7,8 @@ elgg.security.token = {}; elgg.security.tokenRefreshFailed = false; +elgg.security.tokenRefreshTimer = null; + /** * Sets the currently active security token and updates all forms and links on the current page. * @@ -30,31 +32,17 @@ elgg.security.setToken = function(json) { }; /** - * Security tokens time out, so lets refresh those every so often. + * Security tokens time out so we refresh those every so often. * - * @todo handle error and bad return data + * @private */ elgg.security.refreshToken = function() { elgg.action('security/refreshtoken', function(data) { - - // @todo might want to move this to setToken() once http://trac.elgg.org/ticket/3127 - // is implemented. It's here right now to avoid soggy code. - if (!data || !(data.output.__elgg_ts && data.output.__elgg_token)) { - elgg.register_error(elgg.echo('js:security:token_refresh_failed', [elgg.get_site_url()])); - elgg.security.tokenRefreshFailed = true; - - // don't setToken because we refresh every 5 minutes and tokens are good for 1 - // hour by default - return; - } - - // if had problems last time, let them know it's working now - if (elgg.security.tokenRefreshFailed) { - elgg.system_message(elgg.echo('js:security:token_refreshed', [elgg.get_site_url()])); - elgg.security.tokenRefreshFailed = false; + if (data && data.output.__elgg_ts && data.output.__elgg_token) { + elgg.security.setToken(data.output); + } else { + clearInterval(elgg.security.tokenRefreshTimer); } - - elgg.security.setToken(data.output); }); }; @@ -112,9 +100,8 @@ elgg.security.addToken = function(data) { }; elgg.security.init = function() { - //refresh security token every 5 minutes - //this is set in the js/elgg PHP view. - setInterval(elgg.security.refreshToken, elgg.security.interval); + // elgg.security.interval is set in the js/elgg PHP view. + elgg.security.tokenRefreshTimer = setInterval(elgg.security.refreshToken, 60 * 1000); }; elgg.register_hook_handler('boot', 'system', elgg.security.init); \ No newline at end of file diff --git a/languages/en.php b/languages/en.php index 353896047..fe450b8a2 100644 --- a/languages/en.php +++ b/languages/en.php @@ -1189,7 +1189,7 @@ You cannot reply to this email.", * Action gatekeeper */ 'actiongatekeeper:missingfields' => 'Form is missing __token or __ts fields', - 'actiongatekeeper:tokeninvalid' => "We encountered an error (token mismatch). This probably means that the page you were using expired.", + 'actiongatekeeper:tokeninvalid' => "The page you were using had expired. Please try again.", 'actiongatekeeper:timeerror' => 'The page you were using has expired. Please refresh and try again.', 'actiongatekeeper:pluginprevents' => 'A extension has prevented this form from being submitted.', 'actiongatekeeper:uploadexceeded' => 'The size of file(s) uploaded exceeded the limit set by your site administrator', @@ -1211,7 +1211,7 @@ You cannot reply to this email.", * Javascript */ - 'js:security:token_refresh_failed' => 'Cannot contact %s. You may experience problems saving content.', + 'js:security:token_refresh_failed' => 'Failed to contact %s. You may experience problems saving content. Please refresh this page.', 'js:security:token_refreshed' => 'Connection to %s restored!', /** -- cgit v1.2.3 From 8a76e62accd81a68724b424a77421dac7a9d9a12 Mon Sep 17 00:00:00 2001 From: cash Date: Tue, 5 Mar 2013 17:21:04 -0500 Subject: fixed token refresh interval - introduced during development for this release --- js/lib/security.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'js') diff --git a/js/lib/security.js b/js/lib/security.js index af02824a6..9c12f8586 100644 --- a/js/lib/security.js +++ b/js/lib/security.js @@ -101,7 +101,7 @@ elgg.security.addToken = function(data) { elgg.security.init = function() { // elgg.security.interval is set in the js/elgg PHP view. - elgg.security.tokenRefreshTimer = setInterval(elgg.security.refreshToken, 60 * 1000); + elgg.security.tokenRefreshTimer = setInterval(elgg.security.refreshToken, elgg.security.interval); }; elgg.register_hook_handler('boot', 'system', elgg.security.init); \ No newline at end of file -- cgit v1.2.3 From 42c52d51ea0e9047134441b31c8a9c7eac7922ef Mon Sep 17 00:00:00 2001 From: cash Date: Sun, 10 Mar 2013 17:52:38 -0400 Subject: fixing line endings --- js/lib/ui.river.js | 26 ++--- .../theme_preview/components/image_block.php | 12 +-- .../default/theme_preview/components/list.php | 38 +++---- .../default/theme_preview/components/messages.php | 10 +- .../default/theme_preview/components/table.php | 22 ++-- .../views/default/theme_preview/icons/avatars.php | 72 ++++++------- .../views/default/theme_preview/icons/sprites.php | 120 ++++++++++----------- .../default/theme_preview/modules/modules.php | 44 ++++---- .../theme_preview/navigation/breadcrumbs.php | 20 ++-- .../default/theme_preview/navigation/default.php | 22 ++-- .../default/theme_preview/navigation/extras.php | 34 +++--- .../default/theme_preview/navigation/filter.php | 26 ++--- .../theme_preview/navigation/horizontal.php | 24 ++--- .../theme_preview/navigation/owner_block.php | 26 ++--- .../default/theme_preview/navigation/page.php | 38 +++---- .../theme_preview/navigation/pagination.php | 14 +-- .../default/theme_preview/navigation/site.php | 36 +++---- .../default/theme_preview/navigation/tabs.php | 18 ++-- .../default/theme_preview/typography/headings.php | 10 +- .../default/theme_preview/typography/misc.php | 32 +++--- .../default/theme_preview/typography/paragraph.php | 36 +++---- 21 files changed, 340 insertions(+), 340 deletions(-) (limited to 'js') diff --git a/js/lib/ui.river.js b/js/lib/ui.river.js index a56a664a4..c103fabb3 100644 --- a/js/lib/ui.river.js +++ b/js/lib/ui.river.js @@ -1,14 +1,14 @@ -elgg.provide('elgg.ui.river'); - -elgg.ui.river.init = function() { - $('#elgg-river-selector').change(function() { - var url = window.location.href; - if (window.location.search.length) { - url = url.substring(0, url.indexOf('?')); - } - url += '?' + $(this).val(); - elgg.forward(url); - }); -}; - +elgg.provide('elgg.ui.river'); + +elgg.ui.river.init = function() { + $('#elgg-river-selector').change(function() { + var url = window.location.href; + if (window.location.search.length) { + url = url.substring(0, url.indexOf('?')); + } + url += '?' + $(this).val(); + elgg.forward(url); + }); +}; + elgg.register_hook_handler('init', 'system', elgg.ui.river.init); \ No newline at end of file diff --git a/mod/developers/views/default/theme_preview/components/image_block.php b/mod/developers/views/default/theme_preview/components/image_block.php index 0bb16428b..ecd35ac65 100644 --- a/mod/developers/views/default/theme_preview/components/image_block.php +++ b/mod/developers/views/default/theme_preview/components/image_block.php @@ -1,6 +1,6 @@ -title = "Object 1"; -$obj1->description = $ipsum; - -$obj2 = new ElggObject(); -$obj2->title = "Object 2"; -$obj2->description = $ipsum; - -$obj3 = new ElggObject(); -$obj3->title = "Object 3"; -$obj3->description = $ipsum; - -$obj4 = new ElggObject(); -$obj4->title = "Object 4"; -$obj4->description = $ipsum; - -echo elgg_view('page/components/list', array('items' => array($obj1, $obj2, $obj3, $obj4))); +title = "Object 1"; +$obj1->description = $ipsum; + +$obj2 = new ElggObject(); +$obj2->title = "Object 2"; +$obj2->description = $ipsum; + +$obj3 = new ElggObject(); +$obj3->title = "Object 3"; +$obj3->description = $ipsum; + +$obj4 = new ElggObject(); +$obj4->title = "Object 4"; +$obj4->description = $ipsum; + +echo elgg_view('page/components/list', array('items' => array($obj1, $obj2, $obj3, $obj4))); diff --git a/mod/developers/views/default/theme_preview/components/messages.php b/mod/developers/views/default/theme_preview/components/messages.php index ac4d2bfd7..a53255291 100644 --- a/mod/developers/views/default/theme_preview/components/messages.php +++ b/mod/developers/views/default/theme_preview/components/messages.php @@ -1,5 +1,5 @@ -
    -
  • Success message (.elgg-state-success)
  • -
  • Error message (.elgg-state-error)
  • -
  • Notice message (.elgg-state-notice)
  • -
+
    +
  • Success message (.elgg-state-success)
  • +
  • Error message (.elgg-state-error)
  • +
  • Notice message (.elgg-state-notice)
  • +
diff --git a/mod/developers/views/default/theme_preview/components/table.php b/mod/developers/views/default/theme_preview/components/table.php index 8b8b13e76..7d619dcea 100644 --- a/mod/developers/views/default/theme_preview/components/table.php +++ b/mod/developers/views/default/theme_preview/components/table.php @@ -1,12 +1,12 @@ - -"; - for ($i = 1; $i < 5; $i++) { - echo ''; - for ($j = 1; $j < 3; $j++) { - echo ""; - } - echo ''; - } -?> +
column 1column 2
value $j
+"; + for ($i = 1; $i < 5; $i++) { + echo ''; + for ($j = 1; $j < 3; $j++) { + echo ""; + } + echo ''; + } +?>
column 1column 2
value $j
\ No newline at end of file diff --git a/mod/developers/views/default/theme_preview/icons/avatars.php b/mod/developers/views/default/theme_preview/icons/avatars.php index f50a6b70d..3aa1eda26 100644 --- a/mod/developers/views/default/theme_preview/icons/avatars.php +++ b/mod/developers/views/default/theme_preview/icons/avatars.php @@ -1,36 +1,36 @@ - - - - - $size"; - } - ?> - - - - '; - echo elgg_view_entity_icon($user, $size, array('use_hover' => false)); - echo ''; - } - ?> - - - - '; - echo elgg_view_entity_icon($group, $size, array('use_hover' => false)); - echo ''; - } - ?> - -
User
Group
+ + + + + $size"; + } + ?> + + + + '; + echo elgg_view_entity_icon($user, $size, array('use_hover' => false)); + echo ''; + } + ?> + + + + '; + echo elgg_view_entity_icon($group, $size, array('use_hover' => false)); + echo ''; + } + ?> + +
User
Group
diff --git a/mod/developers/views/default/theme_preview/icons/sprites.php b/mod/developers/views/default/theme_preview/icons/sprites.php index 134dd9aca..3edb0bd7c 100644 --- a/mod/developers/views/default/theme_preview/icons/sprites.php +++ b/mod/developers/views/default/theme_preview/icons/sprites.php @@ -1,61 +1,61 @@ - - -