fix error messages for selector and sitemap ids (#144)

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "web-scraper-chrome-extension",
-	"version": "0.4.13",
+	"version": "0.4.14",
 	"description": "Web data extraction tool implemented as chrome extension",
 	"scripts": {
 		"lint": "eslint --ext .js src",

src/_locales/en/messages.json

@@ -256,6 +256,12 @@
 	},
 	"selecting_file_error": { "message": "Please select a file!" },
 	"sitemap_valid_message": { "message": "Valid sitemap" },
+	"selectorid_empty_message": {
+		"message": "The selector id is required and cannot be empty"
+	},
+	"selectorid_short_message": {
+		"message": "The selector id should be at least 3 characters long"
+	},
 	"selectorid_underscore": { "message": "Selector id cannot start with an underscore _" },
 	"selectorid_is_already_exist": {
 		"message": "Current selector ID is already exist"

src/_locales/ru/messages.json

@@ -256,7 +256,7 @@
 	},
 	"sitemapid_repeated_id": { "message": "Карта обхода с таким идентификатором уже существует" },
 	"sitemapid_empty_message": {
-		"message": "Идентификатор селектора обязателен и не может быть пустым"
+		"message": "Идентификатор карты обхода обязателен и не может быть пустым"
 	},
 	"sitemapurl_empty_message": { "message": "Стартовый URL обязателен и не может быть пустым" },
 	"sitemapurl_invalid_message": {
@@ -276,6 +276,12 @@
 	},
 	"selecting_file_error": { "message": "Пожалуйста, выберите файл!" },
 	"sitemap_valid_message": { "message": "Правильная карта обхода" },
+	"selectorid_empty_message": {
+		"message": "Идентификатор селектора обязателен и не может быть пустым"
+	},
+	"selectorid_short_message": {
+		"message": "Длина идентификатора селектора должна быть как минимум 3 символа"
+	},
 	"selectorid_underscore": {
 		"message": "Идентификатор не может начинаться с нижнего подчеркивания _"
 	},

src/scripts/Controller.js

@@ -16,10 +16,10 @@ import SelectorTable from './Selector/SelectorTable';
 import Model from './Model';
 import Translator from './Translator';
 
-export const SITEMAP_ID_REGEXP = /^[a-z][a-z0-9_\$\(\)\+\-]+$/
+export const SITEMAP_ID_REGEXP = /^[a-z][a-z0-9_\$\(\)\+\-]+$/;
 
-export default class SitemapController {	
-	constructor(store, templateDir) {		
+export default class SitemapController {
+	constructor(store, templateDir) {
 		this.store = store;
 		this.templateDir = templateDir;
 		this.contentScript = getContentScript('DevTools');
@@ -599,9 +599,7 @@ export default class SitemapController {
 													),
 											};
 										}
-										if (
-											!sitemap._id.match(SITEMAP_ID_REGEXP)
-										) {
+										if (!sitemap._id.match(SITEMAP_ID_REGEXP)) {
 											return {
 												valid: false,
 												message:
@@ -1070,11 +1068,11 @@ export default class SitemapController {
 				id: {
 					validators: {
 						notEmpty: {
-							message: Translator.getTranslationByKey('sitemapid_empty_message'),
+							message: Translator.getTranslationByKey('selectorid_empty_message'),
 						},
 						stringLength: {
 							min: 3,
-							message: Translator.getTranslationByKey('sitemapid_short_message'),
+							message: Translator.getTranslationByKey('selectorid_short_message'),
 						},
 						regexp: {
 							regexp: /^[^_].*$/,