From 28e6ae8a31f2f72b9cdfb6ecda883cd58248e89c Mon Sep 17 00:00:00 2001
From: Juliano Maciel <julianomfferreira@gmail.com>
Date: Tue, 11 Dec 2018 11:41:52 -0200
Subject: [PATCH] =?UTF-8?q?Corrigir=20grafia=20"1=20peri=C3=B3dicos"=20#58?=
 =?UTF-8?q?.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 application/config/autoload.php               |  2 +-
 application/helpers/pluralize_helper.php      | 46 +++++++++++++++++++
 application/language/english/scielo_lang.php  |  1 +
 .../portuguese-brazilian/scielo_lang.php      |  1 +
 application/language/spanish/scielo_lang.php  |  1 +
 application/views/partials/collections.php    |  8 ++--
 .../views/partials/journals-letter-filter.php |  5 +-
 .../views/partials/journals-search-form.php   |  3 ++
 8 files changed, 61 insertions(+), 6 deletions(-)
 create mode 100644 application/helpers/pluralize_helper.php

diff --git a/application/config/autoload.php b/application/config/autoload.php
index 6deb420..e83be03 100644
--- a/application/config/autoload.php
+++ b/application/config/autoload.php
@@ -89,7 +89,7 @@
 |
 |	$autoload['helper'] = array('url', 'file');
  */
-$autoload['helper'] = array('cookie', 'url', 'assets', 'text', 'twitter', 'language', 'pagination', 'phpmailer');
+$autoload['helper'] = array('cookie', 'url', 'assets', 'text', 'twitter', 'language', 'pagination', 'phpmailer', 'pluralize');
 
 /*
 | -------------------------------------------------------------------
diff --git a/application/helpers/pluralize_helper.php b/application/helpers/pluralize_helper.php
new file mode 100644
index 0000000..3cd0ed1
--- /dev/null
+++ b/application/helpers/pluralize_helper.php
@@ -0,0 +1,46 @@
+<?php
+
+/**
+ * @author
+ * SciELO - Scientific Electronic Library Online 
+ * @link 
+ * https://www.scielo.org/
+ * @license
+ * Copyright SciELO All Rights Reserved.
+ */
+
+defined('BASEPATH') or exit('No direct script access allowed');
+
+if (!function_exists('pluralize')) {
+
+    /**
+     * Pluralizes a word if quantity is not one.
+     * Based on a stackoverflow response https://stackoverflow.com/questions/1534127/pluralize-in-php.
+     *
+     * @param int $quantity Number of items
+     * @param string $singular Singular form of word
+     * @param string $plural Plural form of word; function will attempt to deduce plural form from singular if not provided
+     * @return string Pluralized word if quantity is not one, otherwise singular
+     */
+    function pluralize($quantity, $singular, $plural = null)
+    {
+        if ($quantity == 1 || !strlen($singular)) {
+            return $singular;
+        }
+
+        if ($plural !== null) {
+            return $plural;
+        }
+
+        $last_letter = strtolower($singular[strlen($singular) - 1]);
+
+        switch ($last_letter) {
+            case 'y':
+                return substr($singular, 0, -1) . 'ies';
+            case 's':
+                return $singular . 'es';
+            default:
+                return $singular . 's';
+        }
+    }
+}
diff --git a/application/language/english/scielo_lang.php b/application/language/english/scielo_lang.php
index 54bbe0c..89c37a0 100644
--- a/application/language/english/scielo_lang.php
+++ b/application/language/english/scielo_lang.php
@@ -25,6 +25,7 @@
 $lang['titles'] = 'titles';
 $lang['articles'] = 'articles';
 $lang['open_access'] = 'in open access';
+$lang['journals_singular'] = 'journal';
 $lang['journals'] = 'journals';
 $lang['books'] = 'Books';
 $lang['journals_list'] = 'Journals';
diff --git a/application/language/portuguese-brazilian/scielo_lang.php b/application/language/portuguese-brazilian/scielo_lang.php
index fc4830a..bb50125 100644
--- a/application/language/portuguese-brazilian/scielo_lang.php
+++ b/application/language/portuguese-brazilian/scielo_lang.php
@@ -25,6 +25,7 @@
 $lang['titles'] = 'títulos';
 $lang['articles'] = 'artigos';
 $lang['open_access'] = 'em acesso aberto';
+$lang['journals_singular'] = 'periódico';
 $lang['journals'] = 'periódicos';
 $lang['books'] = 'Livros';
 $lang['journals_list'] = 'Periódicos';
diff --git a/application/language/spanish/scielo_lang.php b/application/language/spanish/scielo_lang.php
index 4c0e006..f4fee7b 100644
--- a/application/language/spanish/scielo_lang.php
+++ b/application/language/spanish/scielo_lang.php
@@ -25,6 +25,7 @@
 $lang['titles'] = 'títulos';
 $lang['articles'] = 'artículos';
 $lang['open_access'] = 'en acceso abierto';
+$lang['journals_singular'] = 'revista';
 $lang['journals'] = 'revistas';
 $lang['books'] = 'Libros';
 $lang['journals_list'] = 'Revistas';
diff --git a/application/views/partials/collections.php b/application/views/partials/collections.php
index 5d02549..4ed1e70 100644
--- a/application/views/partials/collections.php
+++ b/application/views/partials/collections.php
@@ -20,7 +20,7 @@
                 <?php endif;?>
                 <a href="http://<?= $journal->domain ?>">
                     <h4><?= $journal->name[$lang_index] ?></h4>
-                    <span><?php if ($has_journal_count) : ?><?= $journal->journal_count['current'] ?> <?= lang('journals') ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $journal->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
+                    <span><?php if ($has_journal_count) : ?><?= $journal->journal_count['current'] ?> <?= pluralize($journal->journal_count['current'], lang('journals_singular'), lang('journals')) ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $journal->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
                 </a>
             </dd>
             <?php endforeach; ?>
@@ -38,7 +38,7 @@
                     <?php endif;?>
                     <a href="http://<?= $development->domain ?>">
                         <h4><?= $development->name[$lang_index] ?></h4>
-                        <span><?php if ($has_journal_count) : ?><?= $development->journal_count['current'] ?> <?= lang('journals') ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $development->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
+                        <span><?php if ($has_journal_count) : ?><?= $development->journal_count['current'] ?> <?= pluralize($development->journal_count['current'], lang('journals_singular'), lang('journals')) ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $development->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
                     </a>
                 </dd>
             <?php endforeach; ?>
@@ -56,7 +56,7 @@
                     <?php endif;?>
                     <a href="http://<?= $discontinued->domain ?>">
                         <h4><?= $discontinued->name[$lang_index] ?></h4>
-                        <span><?php if ($has_journal_count) : ?><?= $discontinued->journal_count['deceased'] ?> <?= lang('journals') ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $discontinued->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
+                        <span><?php if ($has_journal_count) : ?><?= $discontinued->journal_count['deceased'] ?> <?= pluralize($discontinued->journal_count['deceased'], lang('journals_singular'), lang('journals')) ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $discontinued->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
                     </a>
                 </dd>
             <?php endforeach; ?>
@@ -88,7 +88,7 @@
                     <?php endif;?>
                     <a href="http://<?= $other->domain ?>">
                         <h4><?= $other->name[$lang_index] ?></h4>
-                        <span><?php if ($has_journal_count) : ?><?= $other->journal_count['current'] ?> <?= lang('journals') ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $other->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
+                        <span><?php if ($has_journal_count) : ?><?= $other->journal_count['current'] ?> <?= pluralize($other->journal_count['current'], lang('journals_singular'), lang('journals')) ?> •<?php endif; ?> <?php if ($has_document_count) : ?><?= $other->document_count ?> <?= lang('articles') ?><?php endif; ?></span>
                     </a>
                 </dd>
             <?php endforeach; ?>
diff --git a/application/views/partials/journals-letter-filter.php b/application/views/partials/journals-letter-filter.php
index 79a1fe7..9f836f7 100644
--- a/application/views/partials/journals-letter-filter.php
+++ b/application/views/partials/journals-letter-filter.php
@@ -1,3 +1,6 @@
+<?php
+defined('BASEPATH') or exit('No direct script access allowed');
+?>
 <div class="btn-group-vertical btn-group-fixed" role="group" aria-label="...">
     <?=create_letter_filter(current_url())?>
-</div>
\ No newline at end of file
+</div>
diff --git a/application/views/partials/journals-search-form.php b/application/views/partials/journals-search-form.php
index 69b1be9..041fc42 100644
--- a/application/views/partials/journals-search-form.php
+++ b/application/views/partials/journals-search-form.php
@@ -1,3 +1,6 @@
+<?php
+defined('BASEPATH') or exit('No direct script access allowed');
+?>
 <form action="<?=current_url()?>">
     <input type="hidden" name="limit" value="<?=$this->input->get('limit', true)?>">
     <input type="hidden" name="letter" value="<?=$this->input->get('letter', true)?>">