вот посмотри эта функция делает то же, но читается лучше и без лишних переменных:
```js
function openPopupFromLink(link) {
	var href = link.getAttribute("href"),
		message=;
    createPopup(
        link.dataset.title,
        link.dataset.message.replace(/%s/g, href),
        function() {
            return location.assign(href);
	});
}
```