Error(s) in GoFish5

[KoenP]

Hi,

Since a several days my GoFish5 start producing errors. It started with a total 'hang' - nothing acted anymore, only Ctrl-Alt-Delete could help me, when I entered '/2' (without the quotes) into the search box .
After that I now get error:

Message:      SQL: ORDER BY clause is invalid.
Command:      Select * From (Select * From (lcSearchEngineResultsAlias) Order By &lcOrderBy) source Into Cursor gf_temp_csr ReadWrite
Method:       GOFISH.FILLSEARCHRESULTSCURSOR

Needles to say that I have meanwhile redownloaded and reinstalled GoFish.app today however same effect.

Could this be related to other feautures which we face since the last windows update ( Beaufify works differently for example) ?

Any idea how to cure this?

Regards,

Koen

[Tamar Granor]

Maybe clean out the saved data for GoFish?

Tamar

[KoenP]

Tamar,

how to proceed? What shall I have to clean/remove before I reinstall GoFish?

Koen

Op dinsdag 5 juni 2018 14:32:09 UTC+2 schreef Tamar Granor:

[Jim]

GF creates setting files in Home(7)

Sent from my iPhone

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

[KoenP]

Tamar,

I found them! More than 300(!) directories. What a carbadge. I used Everything to locate them and since they are located in a hidden or protected directory, I cannot delete them all in one go, have to do that directory by directory. Next 24 hours will be busy. Is there not a procedure which could do this work?

Grrr....

Regards,

Koen

Op dinsdag 5 juni 2018 18:35:49 UTC+2 schreef Jim:

[KoenP]

Tamar,

I found the GF_search files are stored in a subdirectory of Home(7) and then each in a directory.

Removed the subdirectory from Home(7) and it works again as a charm.

Thanks a lot for your correct answer.

Regards,

Koen

Op dinsdag 5 juni 2018 19:03:08 UTC+2 schreef KoenP:

[Jim Nelson]

Actually, those sub-directories had search results, not settings.  There is a setting (in Options, I believe) that lets you control how long to keep old search results.

The setting files for GF are all in Home(7), not in any sub-directories.

To unsubscribe from this group and stop receiving emails from it, send an email to foxprogofish+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

[Koen Piller]

Jim,

not that it matters, but settings are saved in Home(7)/GF_Saved_SearchResults/directorywithdatetimename e.g. 20180605203107 at least on my pc. The by you metioned setting I will find, a handy thing to keep your pc clean.

Thanks,

Koen

[Mike Yearwood]

I just ran into this same problem. Koen's solution of removing the home(7) garbage does not work for me.

What is the order by supposed to be?

Thanks

Mike

[Mike Yearwood]

Hey all

This error happens because order by cannot order by a memo field, proccode in this case.

I made a little fix. Hopefully others find this helpful.

Mike

GoFish_results.scx method FillSearchResultsCursor

 

      *TODO: The orderby clause here could fail since some field names were changed

      *      in relase 4.3.015. Need to handle this...

 

*Make a cursor with a proccode that can be ordered, because memo fields cannot.

*!*         Select * From (Select * From (lcSearchEngineResultsAlias) Order By &lcOrderBy) source ;

*!*         Into Cursor gf_temp_csr ReadWrite

lnFields = AFIELDS(laFields,m.lcSearchEngineResultsAlias)

lcFields=laFields[1,1]

FOR lnI = 2 TO lnFields

      m.lcFields = m.lcFields + ","+ALLTRIM(laFields[m.lnI,1])

ENDFOR

m.lcFields =STRTRAN(m.lcFields,",PROCCODE","")

m.lcFields1 = "left(proccode,254) as proccode,"+m.lcFields+",proccode as longproccode"

m.lcFields2 = m.lcFields+",longproccode as proccode"

 

SELECT &lcFields1 FROM(m.lcSearchEngineResultsAlias) INTO CURSOR (m.lcSearchEngineResultsAlias)readwrite

 

SELECT &lcFields2 From (Select * From(m.lcSearchEngineResultsAlias) Order By &lcOrderBy) source ;

      Into Cursor gf_temp_csrReadWrite

--

[Mike Yearwood]

Hey there

I found a mistake in my code. Here's the update.

*Make a cursor with a proccode that can be ordered.

*!*         Select * From (Select * From (lcSearchEngineResultsAlias) Order By &lcOrderBy) source ;

*!*         Into Cursor gf_temp_csr ReadWrite

lnFields = AFIELDS(laFields,m.lcSearchEngineResultsAlias)

lcFields=laFields[1,1]

FOR lnI = 2 TO lnFields

      m.lcFields = m.lcFields + ","+ALLTRIM(laFields[m.lnI,1])

ENDFOR

m.lcFields =STRTRAN(m.lcFields,",PROCCODE","")

m.lcFields = STRTRAN(m.lcFields,",LONGPROCCODE","")

m.lcFields1 = m.lcFields+",left(proccode,254) as proccode,proccode as longproccode"

m.lcFields2 = m.lcFields+",longproccode as proccode"

 

SELECT &lcFields1 FROM(m.lcSearchEngineResultsAlias) INTO CURSOR (m.lcSearchEngineResultsAlias)readwrite

 

SELECT &lcFields2 From (Select * From(m.lcSearchEngineResultsAlias) Order By &lcOrderBy) source ;

      Into Cursor gf_temp_csrReadWrite

[Koen Piller]

Mike,

That looks like a usefull update, suggest you post it also on the GoFish forum  - https://groups.google.com/forum/?fromgroups&hl=en#!forum/foxprogofish -  in the hope Matt Slay will spot it and recognizes it to be usufull for an update of GoFisch.app.

Koen

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

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

--

You received this message because you are subscribed to the Google Groups "GoFish" group.

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

[Tamar Granor]

I just ran into this on one of my machines today. It's fine on the other. Short of modifying the source myself, is there a way I can fix this on the affected machine?

FWIW, it started when I moved the ProcCode column way to the left. Weirdly, on my other machine, it doesn't show up and I don't think it ever did.

Tamar

On Sunday, May 13, 2018 at 8:20:22 AM UTC-4, KoenP wrote:

[Art Bergquist]

Curious; what is lcOrderBy set to?

---

From: Tamar Granor <ta...@thegranors.com>
To: GoFish <foxpro...@googlegroups.com>
Sent: Wednesday, October 3, 2018 12:53 PM
Subject: {GoFish} Re: Error(s) in GoFish5

--

You received this message because you are subscribed to the Google Groups "GoFish" group.

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

[Tamar Granor]

PROCCODE ASC, PROCESS ASC, TRIMMEDMATCHLINE ASC, filename, class, name, methodname, MatchStart

 

I know from Mike’s comments that the issue is PROCCODE, but I don’t know why it’s suddenly there.

[Tamar Granor]

Just implemented Mike's solution and it fixed my problem. Someone should apply it to the main source.

One minor bug in Mike's stuff. In the very last line, there's a space missing between the cursor name and READWRITE.

Tamar

[Koen Piller]

Goodmorning,

I cannot find Mike’s solution in this thread, could you please publish it?

Regards

Koen

Op di 9 okt. 2018 om 22:14 schreef Tamar Granor <ta...@thegranors.com>

Just implemented Mike's solution and it fixed my problem. Someone should apply it to the main source.

One minor bug in Mike's stuff. In the very last line, there's a space missing between the cursor name and READWRITE.

Tamar

--

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

--

Verzonden vanaf mijn iPhone

[Tamar Granor]

Koen - look for the posts from Mike Yearwood in this thread on June 19 and 20.

Tamar