Browse Source

Deprecating the %page% style placeholders in PaginatorHelper.
Updating scaffold and bake templates to not use deprecated placeholders.

mark_story 14 years ago
parent
commit
3d44489b0c

+ 1 - 1
lib/Cake/Console/Templates/default/views/index.ctp

@@ -59,7 +59,7 @@
 	<p>
 	<?php echo "<?php
 	echo \$this->Paginator->counter(array(
-	'format' => __('Page %page% of %pages%, showing %current% records out of %count% total, starting on record %start%, ending on %end%')
+	'format' => __('Page {:page} of {:pages}, showing {:current} records out of {:count} total, starting on record {:start}, ending on {:end}')
 	));
 	?>";?>
 	</p>

+ 3 - 1
lib/Cake/View/Helper/PaginatorHelper.php

@@ -538,12 +538,14 @@ class PaginatorHelper extends AppHelper {
  * - `model` The model to use, defaults to PaginatorHelper::defaultModel();
  * - `format` The format string you want to use, defaults to 'pages' Which generates output like '1 of 5'
  *    set to 'range' to generate output like '1 - 3 of 13'.  Can also be set to a custom string, containing
- *    the following placeholders `%page%`, `%pages%`, `%current%`, `%count%`, `%model%`, `%start%`, `%end%` and any
+ *    the following placeholders `{:page}`, `{:pages}`, `{:current}`, `{:count}`, `{:model}`, `{:start}`, `{:end}` and any
  *    custom content you would like.
  * - `separator` The separator string to use, default to ' of '
  *
+ * The `%page%` style placeholders also work, but are deprecated and will be removed in a future version.
  * @param mixed $options Options for the counter string. See #options for list of keys.
  * @return string Counter string.
+ * @deprecated The %page% style placeholders are deprecated.
  */
 	public function counter($options = array()) {
 		if (is_string($options)) {

+ 1 - 1
lib/Cake/View/Scaffolds/index.ctp

@@ -63,7 +63,7 @@ endforeach;
 </table>
 	<p><?php
 	echo $this->Paginator->counter(array(
-		'format' => __d('cake', 'Page %page% of %pages%, showing %current% records out of %count% total, starting on record %start%, ending on %end%')
+		'format' => __d('cake', 'Page {:page} of {:pages}, showing {:current} records out of {:count} total, starting on record {:start}, ending on {:end}')
 	));
 	?></p>
 	<div class="paging">