Ticket #1262: zs15_fbennett_2008-12-18-1.patch

content/zotero/xpcom/csl.js

@@ -21 +21 @@
 */
 
 /*
+ * PATCHES: fbennett
+ *
+ * This version of cls.js reflects the following modification:
+ *
+ * (1) Fix bug in handling of label pluralizations with et al.:
+ *       Currently, the pluralization of labels used with names is
+ *     determined by the number of names in the formatted string.
+ *     For styles that call for showing only one name with et al.,
+ *     this causes a singular label to be used in this case
+ *     (i.e. "John Doe et al. ed." rather than "John Doe et al. eds.").
+ *
+ *     The fbennett (1) change below addresses this issue.
+ */
+
+
+/*
  * CSL: a class for creating bibliographies from CSL files
  * this is abstracted as a separate class for the benefit of anyone who doesn't
  * want to use the Scholar data model, but does want to use CSL in JavaScript
@@ -694 +710 @@
                                                 }
                                         } else if(formattedString.format != "Sort" && 
                                                         name == "label" && variables[j] != "author") {
-                                                newString.append(this._getTerm(variables[j], (maxCreators != 1), child["@form"].toString(), child["@include-period"] == "true"), child);
+/*
+ * fbennett (1): Force names with et al. to plural
+ */
+                                               newString.append(this._getTerm(variables[j], (maxCreators != 1 || useEtAl), child["@form"].toString(), child["@include-period"] == "true"), child);
+/*
+ * fbennett: end
+ */
                                         }
                                 }
                                 success = true;