[Fix] Grumphp always returns success because does not receive the diff files:

- stdin needs to be passed inside the docker command because docker does not get previous stdin if not running using -i.
- We need -t for the message colors but we cannot use -ti because we cannot use TTY here
- If using -t but not -i, we need to pass the stdin as variable to be used inside the docker command

Author: jalogut

config/grumphp/hooks/docker/commit-msg

@@ -22,4 +22,5 @@ DOCKER_COMPOSE_FILE=${DOCKER_COMPOSE_DIR}/docker-compose.yml
 DOCKER_PHP_CONTAINER_ID=$(docker-compose -f ${DOCKER_COMPOSE_FILE} ps -q phpfpm)
 
 # Run GrumPHP
-(cd "${HOOK_EXEC_PATH}" && printf "%s\n" "${DIFF}" | docker exec -t ${DOCKER_PHP_CONTAINER_ID} $(HOOK_COMMAND) "--git-user=$GIT_USER" "--git-email=$GIT_EMAIL" "$COMMIT_MSG_FILE")
+DIFF_STDIN=$(cd "${HOOK_EXEC_PATH}" && printf "%s\n" "${DIFF}")
+docker exec -t ${DOCKER_PHP_CONTAINER_ID} sh -c "echo '${DIFF_STDIN}' | $(HOOK_COMMAND) '--git-user=${GIT_USER}' '--git-email=${GIT_EMAIL}' '${COMMIT_MSG_FILE}'"

config/grumphp/hooks/docker/pre-commit

@@ -18,4 +18,5 @@ DOCKER_COMPOSE_FILE=${DOCKER_COMPOSE_DIR}/docker-compose.yml
 DOCKER_PHP_CONTAINER_ID=$(docker-compose -f ${DOCKER_COMPOSE_FILE} ps -q phpfpm)
 
 # Run GrumPHP
-(cd "${HOOK_EXEC_PATH}" && printf "%s\n" "${DIFF}" | docker exec -t ${DOCKER_PHP_CONTAINER_ID} $(HOOK_COMMAND) '--skip-success-output')
+DIFF_STDIN=$(cd "${HOOK_EXEC_PATH}" && printf "%s\n" "${DIFF}")
+docker exec -t ${DOCKER_PHP_CONTAINER_ID} sh -c "echo '${DIFF_STDIN}' | $(HOOK_COMMAND) '--skip-success-output'"