Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The update of Thor and the tools I use hung up today and would not complete #281

Closed
GTGeek88 opened this issue Dec 6, 2024 · 13 comments
Closed

Comments

@GTGeek88
Copy link

GTGeek88 commented Dec 6, 2024

Not sure what else to say. I just chose Check For Updates from the Thor menu and let it rip. Did fine until seemingly right at the very end. Got the hourglass cursor that wouldn't go away and what you see in the screenshot.

@GTGeek88
Copy link
Author

GTGeek88 commented Dec 6, 2024

Had to End Task on VFP, which shutdown immediately. That would seem to indicate VFP was still taking input from Windows, saw the request, and obliged. I restarted VFP, ran the update again, Thor was the only thing that indicated an update was needed and it ran successfully.

@Jimrnelson
Copy link
Collaborator

@GTGeek88

Has this problem occurred again?

@GTGeek88
Copy link
Author

No, but I haven't updated since then. I'll let you know if it happens again. I'm looking to see if the update fixes another error, which was that when I typed in a new method or property, something like thisform.MyNewMethod, then highlighted MyNewMethod and used the Go To Definition in Thor, which should allow you to add the property or method to the form, it was hit or miss whether it would work or not. Sometimes it would, but other times nothing happened and I'd have to add it "manually." So if I still see that, I'll report it separately.

@Jimrnelson
Copy link
Collaborator

If the problem with GoToDefinition occurs again, please report it at https://github.com/VFPX/GoToDefinition/issues with as much information as possible for me to try to replicate it.

@GTGeek88
Copy link
Author

Jim, the problem with Go To Definition intermittently not coming up to let me add a new property or method happened enough that I feel I'd have seen it by now if the latest version had not fixed it. So, preliminarily, at least, I think the problem is fixed or much reduced. I have not seen the problem at all, so hopefully that bug was completely squashed.

@Jimrnelson
Copy link
Collaborator

@GTGeek88

Good news then! I will mark this issue as closed.

Should you see any recurrence, please supply as much info as possible so that I can try to reproduce.

@GTGeek88
Copy link
Author

Can we reopen this? I wondered if I was going to jinx this. I just put this.LastPmtId = this.LastPmtId + 1 in code and double-clicked LastPmtId and chose Go To Definition and nothing happened. Just zilch. Nada. I don't know what else to tell you. Don't know of anything I've done that could affect this.

@Jimrnelson Jimrnelson reopened this Dec 19, 2024
@Jimrnelson
Copy link
Collaborator

I have reopened this issue.

Nothing at all -- not even a message that look s like this?

image

GoToDef should always at least tell you it did not find anything.

Can you attach the vcx or scx where this occurs?

@Jimrnelson
Copy link
Collaborator

@GTGeek88

Can we reopen this? I wondered if I was going to jinx this. I just put this.LastPmtId = this.LastPmtId + 1 in code and double-clicked LastPmtId and chose Go To Definition and nothing happened. Just zilch. Nada. I don't know what else to tell you. Don't know of anything I've done that could affect this.

Do you have PEMEditor open when you do this?

The only effect you can expect when using GoToDef on a property is that it will attempt to go to that property in PEMEditor if you are in a method in a SCX or VCX. In a PRG, this should not do anything.

@GTGeek88
Copy link
Author

No, nothing at all. Definitely no message that looks like your example.

"The only effect you can expect when using GoToDef on a property . . ."

I don't understand. I've added properties many times this way. Anyway, later, after closing the SCX and building and running my program and getting an error that the property did not exist (because I'd stopped to let you know it happened again the forgot to add the property "manually"), I opened the SCX and double-clicked the property name to highlight it and used Go To Definition and the form came right up and I added it. So, go figure. Intermittent problems are always such a joy to track down. And this never happens on just one form or class. It happens on most any form or class. And I still think it has lessened, but I guess it's not fully fixed. I'll keep watching it.

@Jimrnelson
Copy link
Collaborator

Head scratching time, so here are a few stabs in the dark. (I'm a little at a loss to know what to ask)

(1) Does the problem with GTD not working only fail (i.e., do nothing) when you write This.SomeName? That is, when it fails, do other uses of GTD still work?

(2) When it fails on This.SomeName, have you tried to see whether ThisForm.SomeName works?

(3) When it fails, can you use GTD on some other name (maybe some work in a character string) to see if it works ("working", in this sense, might mean displaying a MessageBox -- which disappears after a few seconds).

(4) I am wondering if it's failing because it's not even being called. You can test this by enabling Thor Debug Mode when you are have problems with it:

image

If it IS being called, then you should see this when you try GoToDef:

image

(You'll want to turn off debug mode once you've tried this -- it's really not of any value otherwise.)

@GTGeek88
Copy link
Author

Jim,

It's been mostly good. Quite reliable, actually. But I did have two occasions here recently where it failed. The second one was just now. I had typed in this.SavePayment(), double-clicked SavePayment and selected Go to definition from the menu. Nothing. So I looked back over your last communication and saw that I should try thisform. Well, that did make it work. I can't say it's always about that, because I do try to use "thisform" only when the object is not the form and I use "this" when I'm in a method of the form. So if that were the only cause, I think I'd see it a lot more. But this time, using thisform did the trick.

@Jimrnelson
Copy link
Collaborator

@GTGeek88

I have created an issue for this on the GoToDefinition issues page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants