What if Company is blank and the data is per company? Why cannot we always replace the blank company name with the current company?

This will also slow down the code since we will load the table metadata, which can be slow on the first call. I'm not sure how slow it is after the first call.

This won't help.

The only thing that would help is to try a Company.Get(Company) to know if the Company text that was passed to the procedure is a valid company name.

I could then remove the check for the DataPerCompany.

if not CompanyRecord.Get(Company) then 
    Company := CompanyName(); 

@pri-kise If the company exist or not is a caller's responsibility to determine. This code should not fix it. If you ask for a non-existing company URL, you should get what you ask.

If we have a problem with blank company name, why cannot we replace it with current company? We know that it exists.

Also keep in mind, we can open a bug for a platform to fix GetURL if we have issues with it.
The system could easily be generating the links to the external environment - we generate the first part and then the part is replaced. We need to be careful with our assumptions.

The problem is not the Blank Company Name.

The problem is Company with the value of the label.

CrossCompanyDataLbl: Label '(Cross-Company Data)'

I can change the code to check for the label CrossCompanyDataLbl.
....