Case issues across platforms

JohnMoshakis · June 21, 2020, 5:48pm

I started off with a Island console app with the following

  var response := Http.ExecuteRequestSynchronous(new HttpRequest(Url.UrlWithString('http://api.isailed.com/'),HttpRequestMode.Get));

I then created a cocoa console app and pasted the same code in the main.

When compiling one of the suggestions was to change the case so I ended up with

  var content := Http.ExecuteRequestSynchronous(new HttpRequest(URL.URLWithString('http://api.isailed.com/'),HttpRequestMode.Get));

Shouldnt they both be using Url ?

Cheers,
John

mh · June 21, 2020, 6:46pm

Curious. Elements RTL2 has only

class method UrlWithString(aUrlString: not nullable String): not nullable Url;

so i’m not sure where the other comes from. But also note that the class is called Url`. It seems like you might have different “URL” class in scope from somewhere else — could I see complete test case that shows this?

mh · June 21, 2020, 6:48pm

FWIW, your testcase (which I overlooked first) complies clean here with only two expected warnings; no case mismatch and no auto-fix.

2 Messages
CocoaHttpConsoleApplication: Program.pas, line 12 — (H11) Local variable “content” is assigned to but never read
HttpConsoleApplication: Program.pas, line 9 — (H11) Local variable “response” is assigned to but never read

JohnMoshakis · June 21, 2020, 6:51pm

I checked with goto defintion and I was using Foundation.URL. That also has the same method

mh · June 21, 2020, 6:52pm

That’d explain it. make sure you count have Foundation in the uses after Elements.RTL. This will not be the only conflict, otherwise. Odd that your test case didn’t have that though? Do you see the happening with the exact project you sent?

JohnMoshakis · June 21, 2020, 6:55pm

What I sent you was after I built so it was already applied.

mh · June 21, 2020, 8:33pm

No, what you sent had it lowercase and no warning.

JohnMoshakis · June 21, 2020, 8:43pm

First screenshot, do a build and then i get the second screenshot

mh · June 21, 2020, 9:11pm

Ah yes never mind, I misread and thought the Island one did then case warning. The Toffee one was uppercased, yes.