Bug in GoFish 5.0.156

[Mike Potjer]

Hi Jim,

I'm getting a "String is too long to fit" error in the code shown in the screenshot below.  The Caption property is limited to 255 characters, but the assembled string in this case is longer than that.  Part of the problem is that the lcMethod variable in this example is being assembled in a way that duplicates part of the object path, making the resulting value considerably larger than it needs to be (see the highlighted section of lcMethod in the screenshot).  Since the error is in a Refresh(), I can't just ignore it to move past the error, but have to cancel GF.  If you need more information, please let me know.  I could probably put together a simple example to reproduce the problem as well, if you need it.

Thanks,

Mike

[Jim Nelson]

Thanks for the error report (I guess).

Looks like something a Try/Catch should handle easily enough (even if I do not understand why that string is so long).

I should have a fix with the next few days.

Sorry about that!

--
You received this message because you are subscribed to the Google Groups "Thor, the Tool Manager for FoxPro" group.
To unsubscribe from this group and stop receiving emails from it, send an email to FoxProThor+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

--

Jim Nelson

(805) 498-9195 (preferred)
(720) 837-3536 (cell)

www.kongcompany.com

[Mike Potjer]

You could also build the caption in a variable, and check its length before assigning it to the Caption property.  Then you wouldn’t need a TRY/CATCH, and you could calculate how much to chop off to make a valid Caption.

 

Mike

 

From: foxpro...@googlegroups.com [mailto:foxpro...@googlegroups.com] On Behalf Of Jim Nelson
Sent: Monday, October 24, 2016 11:26 AM
To: FoxPr...@googlegroups.com
Cc: GoFish <foxpro...@googlegroups.com>
Subject: {GoFish} Re: {Thor} Bug in GoFish 5.0.156

 

Thanks for the error report (I guess).

 

Looks like something a Try/Catch should handle easily enough (even if I do not understand why that string is so long).

 

I should have a fix with the next few days.

 

Sorry about that!

On Mon, Oct 24, 2016 at 8:56 AM, Mike Potjer <pot...@gmail.com> wrote:

Hi Jim,

 

I'm getting a "String is too long to fit" error in the code shown in the screenshot below.  The Caption property is limited to 255 characters, but the assembled string in this case is longer than that.  Part of the problem is that the lcMethod variable in this example is being assembled in a way that duplicates part of the object path, making the resulting value considerably larger than it needs to be (see the highlighted section of lcMethod in the screenshot).  Since the error is in a Refresh(), I can't just ignore it to move past the error, but have to cancel GF.  If you need more information, please let me know.  I could probably put together a simple example to reproduce the problem as well, if you need it.

 

 

Thanks,

Mike

--

You received this message because you are subscribed to the Google Groups "Thor, the Tool Manager for FoxPro" group.

To unsubscribe from this group and stop receiving emails from it, send an email to FoxProThor+...@googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

 

--

Jim Nelson

(805) 498-9195 (preferred)
(720) 837-3536 (cell)

www.kongcompany.com

--
You received this message because you are subscribed to a topic in the Google Groups "GoFish" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/foxprogofish/7NGk_fBZD_0/unsubscribe.
To unsubscribe from this group and all its topics, send an email to foxprogofish...@googlegroups.com.

[Jim Nelson]

Mike --

GF Verison 5.0.157 for ya -- this take care of your problem?

--

You received this message because you are subscribed to the Google Groups "Thor, the Tool Manager for FoxPro" group.

To unsubscribe from this group and stop receiving emails from it, send an email to FoxProThor+unsubscribe@googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

[Mike Potjer]

Yes, thanks!  I did a quick check before installing the update, to confirm where I was getting the error.  After installing the update, it is working without error.

 

Thanks again!

Mike

 

From: foxpro...@googlegroups.com [mailto:foxpro...@googlegroups.com] On Behalf Of Jim Nelson
Sent: Tuesday, October 25, 2016 7:54 PM
To: FoxPr...@googlegroups.com
Cc: GoFish <foxpro...@googlegroups.com>
Subject: {GoFish} Re: {Thor} Bug in GoFish 5.0.156

 

Mike --

 

GF Verison 5.0.157 for ya -- this take care of your problem?

On Mon, Oct 24, 2016 at 7:56 AM, Mike Potjer <pot...@gmail.com> wrote:

Hi Jim,

 

I'm getting a "String is too long to fit" error in the code shown in the screenshot below.  The Caption property is limited to 255 characters, but the assembled string in this case is longer than that.  Part of the problem is that the lcMethod variable in this example is being assembled in a way that duplicates part of the object path, making the resulting value considerably larger than it needs to be (see the highlighted section of lcMethod in the screenshot).  Since the error is in a Refresh(), I can't just ignore it to move past the error, but have to cancel GF.  If you need more information, please let me know.  I could probably put together a simple example to reproduce the problem as well, if you need it.

 

 

Thanks,

Mike

--

You received this message because you are subscribed to the Google Groups "Thor, the Tool Manager for FoxPro" group.

To unsubscribe from this group and stop receiving emails from it, send an email to FoxProThor+...@googlegroups.com.

For more options, visit https://groups.google.com/d/optout.

 

--

Jim Nelson

(805) 498-9195 (preferred)
(720) 837-3536 (cell)

www.kongcompany.com

--

You received this message because you are subscribed to a topic in the Google Groups "GoFish" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/foxprogofish/7NGk_fBZD_0/unsubscribe.
To unsubscribe from this group and all its topics, send an email to foxprogofish...@googlegroups.com.