ad 1) Well in fact CheckIP is not pretty to me. It takes some string and without looking into the code no one knows that the string should be reference (id) of a widget which will be queried for the ip.

ad 2) refactoring? At least in my case it is daily bread of common yast developer ;-)