diff options
| author | Teddy Wing | 2018-04-18 22:02:27 +0200 | 
|---|---|---|
| committer | Teddy Wing | 2018-04-18 22:02:27 +0200 | 
| commit | a2cacedf44d2488528cae45ecc1b48483903be62 (patch) | |
| tree | 1e9a4ca8e96e085d0c53f4a170c66f02d1da57c7 | |
| parent | c3f749355ecde0f1d79524fa9b00f47e32e1437f (diff) | |
| download | redprine-a2cacedf44d2488528cae45ecc1b48483903be62.tar.bz2 | |
update_redmine_statuses(): Don't update if there are no issues
Previously this would run `update_redmine_status()` even if
`$issue_ids_and_urls` was empty, which would make an unnecessary request
without the required data.
| -rwxr-xr-x | redprine | 8 | 
1 files changed, 5 insertions, 3 deletions
| @@ -140,9 +140,11 @@ function issue_numbers_and_pull_request_urls () {  function update_redmine_statuses () {  	local issue_ids_and_urls="$1" -	for id_and_url in "$issue_ids_and_urls"; do -		update_redmine_status "$id_and_url" -	done +	if [ ! -z "$issue_ids_and_urls" ]; then +		for id_and_url in "$issue_ids_and_urls"; do +			update_redmine_status "$id_and_url" +		done +	fi  }  function update_redmine_status () { | 
