Add plural format to views_handler_area_result

+++ b/handlers/views_handler_area_result.incundefined
@@ -87,8 +124,11 @@ class views_handler_area_result extends views_handler_area {
+    if (!empty($this->options['format_plural']) && $replacements[$this->options['format_plural_count']] != 1) {

There so no way to select no plural token so this check seems a bit redundant.

+++ b/handlers/views_handler_area_result.incundefined
@@ -87,8 +124,11 @@ class views_handler_area_result extends views_handler_area {
+    if (!empty($total) || !empty($this->options['empty']) && !empty($this->view->build_info['count_query'])) {

This change seems to be unrelated ...

Patch in #1 applies cleanly and works well for me.

#1 works for me, thanks.

This is a good and useful feature. I vote that it be added to Views.

This is working great for me, recommend merging into Views

Tested on simplytest.me, works great.

Maybe the help-text's could be a bit more clear about the Format plural field being used instead of the default Display Field.
And maybe
Token used to detect the plural format vs.
Token used to select the plural format

See the different Header- text's in the preview-area (darkened by the overlay), singular above, plural below:

+++ b/handlers/views_handler_area_result.inc
@@ -87,8 +124,11 @@ class views_handler_area_result extends views_handler_area {
+    if (!empty($this->options['format_plural']) && $replacements[$this->options['format_plural_count']] != 1) {
...
+    if (!empty($total) || !empty($this->options['empty']) && !empty($this->view->build_info['count_query'])) {

We should have at least some documentation of what is going on.

Works for me, recommend merge.

It sounds like D8 has some neat "true" pluralizing for multilingual as other languages have non-binary decisions for the pluralization.

@MatthewHager We can likely get this in D7 if someone documents those lines as @dawehner mentioned in #10 Maybe you want to give it a try?

This likely couldn't be put in D8 in the same way though...

Added some comments per #10 and #12

Works for me since a year now.

It would be nice to merge this and avoid to reapply patch after each security update.

=> RTBC

Added an updated patch for the new version of views.

Thanks @SaytO. Let's see what testbot says.

This looks pretty good, thank you all for working on it.

My one concern is around this line:

+ && !empty($this->view->build_info['count_query'])


It changes the logic every so slightly and I wonder if it should be handled as a separate request.

Patch #16 by SaytO seems to be working fine on Drupal 7.69 and Views 7.x-3.23
Thanks

Reroll patch to version 7.x-3.24

Patch #20 works with Views 7.x-3.24, thanks.

Patch #20 works for us too (slightly modified for 7.x-3.20), although I'll note that this doesn't help you if you're inserting multiple tokens, each of which can have a plural or singular form. But that would require far more complexity and this meets the large majority of use cases, so it would be great to have it included.

It worked here as well. +1 to it

Thank you so much, everyone.

Moved to the dev branch