Help with a DO loop scripyt

Hey @cgiordano I can offer some advice on this. I would consider rewriting this into a couple of functions within the script and then calling them when needed. You may not be aware of it, but a function can be called or re-run by a result returned within the function itself, which can create the kind of loop you're looking for, I believe.

I took a little time to rewrite your script using just cocoaDialog only, although you could certainly try to incorporate jamfHelper into this if you really want to. I'm also using the last beta 3 version of cocoaDialog, which has some extra features I'm using below, such as value-required and empty-text The value-required flag is used with things like inputbox and other window types to ensure a value is entered or a box selected, etc before confirming with a button. Meaning it's impossible to click OK or Enter and have an empty value in the field. The dialog won't allow it. The empty-text flag is used to customize the sheet that appears when no value is detected.

In the below, you'll see I broke this up into 2 functions. The first one calls the asset tag enter dialog, then checks to see if the value returned is an integer. Here. I'm doing some simple math by dividing the value entered by itself, which will always yield "1" if it's an integer. If that fails, it calls the function again, but in order to make the text appear differently, it sets a loop flag, which gets checked each time the function is run and sets either the error text for the dialog, or the original text. If the value entered is good, it clears the loop flag in preparation for if it gets called again.
From there, it pops up the confirmation dialog, checks for which button was clicked and if the user clicks No, it calls the original function to enter the asset tag again. If they click Yes, it moves onto setting the tag in the ARD field. In this way, you can keep going round and round almost indefinitely. For example, enter "123" then click No on the confirmation, then enter "456" then click No again and enter "789" and so on and so until the integer value is confirmed by clicking "Yes"

I added comments to most items so it's easy (hopefully) to see what's actually going on.
Give this script a try to see how it works for you. I would suggest getting the CD 3 beta version though since I can't predict how it will work under the 2.x release.

#!/bin/bash

## Path to cocoaDialog binary
CD="/Library/Application Support/JAMF/bin/cocoaDialog.app/Contents/MacOS/cocoaDialog"

function checkTag ()
{

## Display dialog asking for confirmation that the tag entered was correct
confirmTag=$("$CD" msgbox 
    --title "Asset Tag Confirmation" 
    --text "Confirm Asset Tag" 
    --informative-text "You said you're asset tag is: $tag

Is that correct?" 
    --button1 "  Yes  " 
    --button2 "  No  " 
    --icon info)

## Check the button clicked
if [ "$confirmTag" == "1" ]; then
    echo "Tag was confirmed by user"

    ## User confirmed. Write the value entered into the ARD info field for the system
    #/System/Library/CoreServices/RemoteManagement/ARDAgent.app/Contents/Resources/kickstart -configure -computerinfo -set1 -1 $tag
elif [ "$confirmTag" == "2" ]; then
    ## User clicked No. Re-run the function to ask for the asset tag
    echo "Re-running enter asset tag"
    enterTag
fi

}


function enterTag ()
{

## Default title string for dialog
Title="Asset Tag"

## If the loop flag is set, set the dialog strings to the error values
if [ "$loop" ]; then
    Text="Incorrect Asset Tag Entered.

Letters are not allowed in your asset tag. Please re-enter your asset tag using only numbers"
else
    ## If the loop flag is NOT set, set the dialog strings to their normal values
    Text="Asset Tag Number Required.

Please enter the asset tag number of your computer"
fi

## Display dialog asking for asset tag value
tag=$("${CD}" inputbox 
    --title "$Title" 
    --text "" 
    --informative-text "$Text" 
    --button1 " Enter " 
    --value-required 
    --empty-text "You must enter a value to continue" 
    --icon computer 
    --quiet)

## Run a math function to determine if the value entered was an integer
## Divides the value by itself, which will always be "1" if an integer, or error if not

check=$(expr $tag / $tag 2>/dev/null)

## Check on the value returned from above
if [[ "$check" == "1" ]]; then
    echo "Integer entered. Continuing."
    ## Blank out the loop flag
    loop=""
    ## Run the checkTag function
    checkTag
else
    echo "Non-integer entered. Re-run function"
    ## Set the loop flag
    loop="Yes"
    ## Run the enterTag function again
    enterTag
fi

}

## Starts here. Run the function asking for the asset tag
enterTag