applyPatch() sometimes returns `false` instead of errors

[saschanaz]

jsdiff/src/patch/apply.js

Lines 72 to 74 in 10aaabb

	if (localOffset === undefined) {
	return false;
	}

This is weird, it should really throw an error, right?

[barisusakli]

I think we are running into this as well with this code.

	data.post.content = data.diffs.reduce(function (content, currentDiff) {
		console.log(typeof content, content, currentDiff);
		const result = diff.applyPatch(content, currentDiff.patch, {
			fuzzFactor: 1,
		});
		
		return result;
	}, data.post.content);

After two iterations the output is this

string <p><img src="/assets/uploads/files/1555828077895-index.png" alt="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" title="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" style="cursor:pointer;" /></p>
<ul><li>Организационно-правовая форма: ООО «Игра-сервис» + ИП Долин Юрий Иванович</li><li>Автономная система: <a href="https://bgp.he.net/AS33991" target="_blank">AS33991</a>.</li><li>Услуги: интернет, телевидение, телефония, видеонаблюдение, IT-сервис.</li><li><a href="https://www.g-service.ru/verify-address/" target="_blank">Проверка адреса</a> подключения.</li><li>Работает в Красноярском крае (Красноярск, Зеленогорск, Назарово, Ужур, Шарыпово) и Ульяновске.</li><li>Интернет-сайт: <a href="https://www.g-service.ru/" target="_blank">g-service.ru</a></li><li>Телефоны службы поддержки: +7 (800) 234−39−13 (все города присутствия, звонок бесплатный), +7 (391) 291−39−13 (Красноярск), +7 (39155) 5−51−00 (Назарово), +7 (8422) 33−20−00 (Ульяновск).</li></ul>
<p><span class="write-container"><br /></span></p>
<p>Минусы:</p>
<ul><li>«верхние» тарифы могли бы быть и пониже;</li><li>не всем по вкусу название;</li></ul>
<p>
</p>
<p>Плюсы:</p>
<ul><li>подключение по IPoE (раздача сетевых параметров по DHCP);</li><li>имеется возможность подключения на тарифы с максимальной скростью  до 500 Мбит/с (в Красноярске);</li><li>работает кое-где в <a href="https://www.g-service.ru/private-sector/" target="_blank">частном секторе</a> (по технологии GPON).</li></ul>
<p>
</p> { patch:
   'Index: \n===================================================================\n--- \n+++ \n@@ -1,7 +1,7 @@\n-<p><img src="/assets/uploads/files/1555828077895-index.png" alt="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" title="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" style="cursor:pointer;"></p>\n-<ul><li>Организационно-правовая форма: ООО «Игра-сервис» + ИП Долин Юрий Иванович</li><li>Автономная&nbsp;система: <a href="https://bgp.he.net/AS33991" target="_blank">AS33991</a>.</li><li>Услуги: интернет, телевидение, телефония, видеонаблюдение, IT-сервис.</li><li><a href="https://www.g-service.ru/verify-address/" target="_blank">Проверка адреса</a> подключения.</li><li>Работает&nbsp;в&nbsp;Красноярском&nbsp;крае&nbsp;(Красноярск, Зеленогорск, Назарово, Ужур, Шарыпово)&nbsp;и&nbsp;Ульяновске.</li><li>Интернет-сайт: <a href="https://www.g-service.ru/" target="_blank">g-service.ru</a></li><li>Телефоны службы поддержки: +7 (800) 234−39−13 (все города присутствия, звонок бесплатный), +7 (391) 291−39−13 (Красноярск), +7 (39155) 5−51−00 (Назарово), +7 (8422) 33−20−00 (Ульяновск).</li></ul>\n-<p><span class="write-container"><br></span></p>\n+<p><img src="/assets/uploads/files/1555828077895-index.png" alt="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" title="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" style="cursor:pointer;" /></p>\n+<ul><li>Организационно-правовая форма: ООО «Игра-сервис» + ИП Долин Юрий Иванович</li><li>Автономная система: <a href="https://bgp.he.net/AS33991" target="_blank">AS33991</a>.</li><li>Услуги: интернет, телевидение, телефония, видеонаблюдение, IT-сервис.</li><li><a href="http://g-service.ru/verify-address/" target="_blank">Проверка адреса</a> подключения.</li><li>Работает в Красноярском крае (Красноярск, Зеленогорск, Назарово, Ужур, Шарыпово) и Ульяновске.</li><li>Интернет-сайт: <a href="http://g-service.ru/" target="_blank">g-service.ru</a></li><li>Телефоны службы поддержки: +7 (800) 234−39−13 (все города присутствия, звонок бесплатный), +7 (391) 291−39−13 (Красноярск), +7 (39155) 5−51−00 (Назарово), +7 (8422) 33−20−00 (Ульяновск).</li></ul>\n+<p><span class="write-container"><br /></span></p>\n <p>Минусы:</p>\n <ul><li>«верхние» тарифы могли бы быть и пониже;</li><li>не всем по вкусу название;</li></ul>\n <p>\n </p>\n',
  pid: '45' }


boolean false { patch:
   'Index: \n===================================================================\n--- \n+++ \n@@ -1,18 +1,4 @@\n-<p><img src="/assets/uploads/files/1555828077895-index.png" alt="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" title="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" style="cursor:pointer;"></p>\n-<ul><li>Организационно-правовая форма: ООО «Игра-сервис» + ИП Долин Юрий Иванович</li><li>Автономная&nbsp;система: <a href="https://bgp.he.net/AS33991" target="_blank">AS33991</a>.</li><li>Услуги: интернет, телевидение, телефония, видеонаблюдение, IT-сервис.</li><li><a href="http://g-service.ru/verify-address/" target="_blank">Проверка адреса</a> подключения.</li><li>Работает&nbsp;в&nbsp;Красноярском&nbsp;крае&nbsp;(Красноярск, Зеленогорск, Назарово, Ужур, Шарыпово)&nbsp;и&nbsp;Ульяновске.</li><li>Интернет-сайт: <a href="http://g-service.ru/" target="_blank">g-service.ru</a></li><li>Телефоны службы поддержки: +7 (800) 234−39−13 (все города присутствия, звонок бесплатный), +7 (391) 291−39−13 (Красноярск), +7 (39155) 5−51−00 (Назарово), +7 (8422) 33−20−00 (Ульяновск).</li></ul>\n-<p><span class="write-container"><br></span></p>\n-<p>Минусы:</p>\n-<ul><li>«верхние» тарифы могли бы быть и пониже;</li><li>не всем по вкусу название;</li></ul>\n-<p>\n-</p>\n-<p>Плюсы:</p>\n-<ul><li>подключение по IPoE (раздача сетевых параметров по DHCP);</li><li>имеется возможность подключения на тарифы с максимальной скростью  до 500 Мбит/с (в Красноярске);</li><li>работает кое-где в <a href="https://www.g-service.ru/private-sector/" target="_blank">частном секторе</a> (по технологии GPON).</li></ul>\n-<p>\n-[poll title="Оцените работу провайдера по пятибалльной шкале"]\n-- 1 &mdash; отвратительно\n-- 2 &mdash; плохо\n-- 3 &mdash; удовлетворительно\n-- 4 &mdash; хорошо\n-- 5 &mdash; отлично\n-- не могу оценить\n-[/poll]</p>\n\\ No newline at end of file\n+<p><img src="/assets/uploads/files/1555828077895-index.png" alt="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" title="Интернет-провайдер «Игра-сервис» («Гейм-сервис», «Г-сервис», «Гей-сервис»)" style="cursor:pointer;" /></p>\n+<ul><li>Организационно-правовая форма: ООО «Игра-сервис» + ИП Долин Юрий Иванович</li><li>Автономная система: <a href="https://bgp.he.net/AS33991" target="_blank">AS33991</a>.</li><li>Услуги: интернет, телевидение, телефония, видеонаблюдение, IT-сервис.</li><li><a href="http://g-service.ru/verify-address/" target="_blank">Проверка адреса</a> подключения.</li><li>Работает в Красноярском крае (Красноярск, Зеленогорск, Назарово, Ужур, Шарыпово) и Ульяновске.</li><li>Интернет-сайт: <a href="http://g-service.ru/" target="_blank">g-service.ru</a></li><li>Телефоны службы поддержки: +7 (800) 234−39−13 (все города присутствия, звонок бесплатный), +7 (391) 291−39−13 (Красноярск), +7 (39155) 5−51−00 (Назарово), +7 (8422) 33−20−00 (Ульяновск).</li></ul><p><span class="write-container"><br /></span></p><p>Минусы:</p><ul><li>«верхние» тарифы могли бы быть и пониже;</li><li>не всем по вкусу название;</li></ul><p>\n+</p><p>Плюсы:</p>\n+<ul><li>подключение по IPoE (раздача сетевых параметров по DHCP);</li><li>имеется возможность подключения на тарифы с максимальной скростью  до 500 Мбит/с (в Красноярске);</li><li>работает кое-где в <a href="https://www.g-service.ru/private-sector/" target="_blank">частном секторе</a> (по технологии GPON).</li></ul>\n\\ No newline at end of file\n',
  pid: '45' }

2019-05-18T09:49:21.398Z [4567/14377] - error: uncaughtException: source.split is not a function

The second call to applyPatch returns boolean false.

[ExplodingCabbage]

Since false here always means the same thing (there was a hunk in the patch that couldn't be fitted to the text to be patched), I think it's basically okay that we just use a boolean return value to signal an error. We should document it, though; the README currently gives no hint that returning false is possible.