Самое первое, что надо сказать:
Не нужно давать временным переменным супер-информативные имена.
- $sqlForSubscriberByLogged - это просто $sql. Эта переменная никогда никому не будет нужна, кроме функции, исполняющей запросы. А ей всё равно, как названа переменная.
- Переменной $queryForSubscriberByLogged в принципе быть не должно, это паразитная, ничего не делающая переменная.
- $fetch_arrayForSubscriberByLogged - это какой-то адов монстр. Нас не должно интересовать, каким способом получена информация, через fetch_array или fetch_row. Нас должно только интересовать её содержимое. Вот его и надо описать в имени переменной. Но тоже без эксцессов.
В итоге должно получиться
$sql = "SELECT id_request_reciver FROM subscribers WHERE id_request_sender = ...";
$SubscriberByLogged = $database->query($sql)->fetch_array();