The only thing I'm questioning is the custom Location field; wondering why that is necessary. Seems like the user could just change the native Location field directly and you could watch for that change to update the lines