Merge branch 'dev-2.0' into addon-fn-proxy

Author: limzykenneth

src/core/internationalization.js

@@ -1,6 +1,7 @@
 import i18next from 'i18next';
 import LanguageDetector from 'i18next-browser-languagedetector';
 import { default as fallbackResources, languages } from '../../translations';
+import { VERSION } from './constants';
 
 if (typeof IS_MINIFIED === 'undefined') {
   // internationalization is only for the unminified build
@@ -147,8 +148,12 @@ export const initialize = () => {
       },
       backend: {
         fallback: 'en',
-        loadPath:
-          'https://cdn.jsdelivr.net/npm/p5/translations/{{lng}}/{{ns}}.json'
+        
+        // ensure that the FES internationalization strings are loaded
+        // from the latest patch of the current minor version of p5.js
+        loadPath: `https://cdn.jsdelivr.net/npm/p5@${
+          VERSION.replace(/^(\d+\.\d+)\.\d+.*$/, '$1')
+        }/translations/{{lng}}/{{ns}}.json`
       },
       partialBundledLanguages: true,
       resources: fallbackResources

src/type/p5.Font.js

@@ -981,10 +981,17 @@ async function create(pInst, name, path, descriptors, rawFont) {
   return new Font(pInst, face, name, path, rawFont);
 }
 
-function createFontFace(name, path, descriptors, rawFont) {
 
-  if (name.includes(' ')) name = "'" + name + "'"; // NOTE: must be single-quotes
+function sanitizeFontName(name) {
+  if (!/^[A-Za-z][A-Za-z0-9_-]*$/.test(name)) {
+    name = "'" + String(name).replace(/'/g, "\\'") + "'";
+  }
+  return name;
+}
+
+function createFontFace(name, path, descriptors, rawFont) {
 
+  name = sanitizeFontName(name);
   let fontArg = rawFont?._compressedData ?? rawFont?._data;
   if (!fontArg) {
     if (!validFontTypesRe.test(path)) {
@@ -1530,6 +1537,7 @@ export const arrayCommandsToObjects = commands => commands.map(command => {
   }
 });
 
+export { sanitizeFontName as _sanitizeFontName };
 export default font;
 
 if (typeof p5 !== 'undefined') {

src/webgl/p5.Camera.js

@@ -3682,8 +3682,7 @@ function camera(p5, fn){
   };
 
   /**
-   * Creates a new <a href="#/p5.Camera">p5.Camera</a> object and sets it
-   * as the current (active) camera.
+   * Creates a new <a href="#/p5.Camera">p5.Camera</a> object.
    *
    * The new camera is initialized with a default position `(0, 0, 800)` and a
    * default perspective projection. Its properties can be controlled with

test/unit/type/p5.Font.js

@@ -1,4 +1,5 @@
 import p5 from '../../../src/app.js';
+import {_sanitizeFontName} from '../../../src/type/p5.Font.js';
 
 suite('p5.Font', function () {
   var myp5;
@@ -61,3 +62,23 @@ suite('p5.Font', function () {
     });
   });
 });
+
+suite('sanitizeFontName', function () {
+  test('fully alphabetic or alpha-leading alnum do not need quotes', function () {
+    assert.equal(_sanitizeFontName('Arial'), 'Arial');
+    assert.equal(_sanitizeFontName('Family900'), 'Family900');
+    assert.equal(_sanitizeFontName('A_b-c'), 'A_b-c');
+  });
+
+  test('names starting with a digit need quotes', function () {
+    assert.equal(_sanitizeFontName('9lives'), "'9lives'");
+  });
+
+  test('names with spaces need quotes', function () {
+    assert.equal(_sanitizeFontName('My Font'), "'My Font'");
+  });
+
+  test('names with commas need quotes', function () {
+    assert.equal(_sanitizeFontName('Foo,Bar'), "'Foo,Bar'");
+  });
+});