Я получаю сообщение об ошибке Uncaught TypeError: Невозможно установить свойство 'onclick' для null

getElementById() поддерживает только один параметр, поэтому вы можете вернуть только один элемент.

Таким образом, либо вы получаете свои элементы один за другим, как это:

document.getElementById('piggy-bank-fb-shear-btn').onclick = function() { // stuff }
document.getElementById('piggy-bank-fb-shear-btn-2').onclick = function() { // stuff }

или вы делаете это через querySelectorAll() который поддерживает не только один параметр.

var elements = document.querySelector('#piggy-bank-fb-shear-btn, #piggy-bank-fb-shear-btn-2');

Но теперь вы не можете просто добавить onclick в конце запроса, потому что querySelectorAll() возвратил несколько элементов, так называемый NodeList. Из-за этого вам нужно будет перебирать elements коллекции, называемые elements:

for (var i = 0; i < elements.length; i++) {
  elements[i].onclick = function() { // stuff }
}

Сразу вы можете получить один элемент, используя document.getElementById. Таким образом, вы можете сохранить id элементов внутри массива и присоединить к нему прослушиватель событий

['piggy-bank-fb-shear-btn', 'piggy-bank-fb-shear-btn-2'].forEach(function(item){
 document.getElementById(item).onclick==//rest of code
})

var clickfnc = function() {

}

document.getElementById('piggy-bank-fb-shear-btn').onclick =clickfnc 
document.getElementById('piggy-bank-fb-shear-btn-2').onclick =clickfnc 

document.getElementById имеет только один параметр, вам нужно изменить это в своем коде:

document.getElementById('piggy-bank-fb-shear-btn').onclick = function;
document.getElementById('piggy-bank-fb-shear-btn-2').onclick = function;


// Maybe you will prefer to use Jquery which you can use:

$('#piggy-bank-fb-shear-btn, #piggy-bank-fb-shear-btn-2').on('click', function() {});