Author Topic: "Find uses Selected Text" broken  (Read 20729 times)

Offline Biplab

  • Developer
  • Lives here!
  • *****
  • Posts: 1874
    • Biplab's Blog
Re: "Find uses Selected Text" broken
« Reply #15 on: May 31, 2007, 07:12:12 pm »
Sorry for this late reply. I'll test it. :)
Be a part of the solution, not a part of the problem.

Offline Biplab

  • Developer
  • Lives here!
  • *****
  • Posts: 1874
    • Biplab's Blog
Re: "Find uses Selected Text" broken
« Reply #16 on: May 31, 2007, 07:52:07 pm »
The uploaded patch fixes the bug. But it introduces other bug.

1. Duplicate entries of previous search texts are created in the list.
Be a part of the solution, not a part of the problem.

Offline Pecan

  • Plugin developer
  • Lives here!
  • ****
  • Posts: 2775
Re: "Find uses Selected Text" broken
« Reply #17 on: May 31, 2007, 08:17:38 pm »
The uploaded patch fixes the bug. But it introduces other bug.

1. Duplicate entries of previous search texts are created in the list.

What list? Are you talking about "find in files" ?

Offline dmoore

  • Developer
  • Lives here!
  • *****
  • Posts: 1576
Re: "Find uses Selected Text" broken
« Reply #18 on: May 31, 2007, 09:15:18 pm »
The uploaded patch fixes the bug. But it introduces other bug.

1. Duplicate entries of previous search texts are created in the list.

What list? Are you talking about "find in files" ?


I assume the list in question is the drop down list of previous searches under "text to search for". I can't replicate.

Offline Pecan

  • Plugin developer
  • Lives here!
  • ****
  • Posts: 2775
Re: "Find uses Selected Text" broken
« Reply #19 on: May 31, 2007, 09:29:59 pm »
And even if there were duplicates in the dropdown list. It appears it's me that put them there and I'd like to keep it that way.

Offline Biplab

  • Developer
  • Lives here!
  • *****
  • Posts: 1874
    • Biplab's Blog
Re: "Find uses Selected Text" broken
« Reply #20 on: June 04, 2007, 06:56:44 pm »
Patch applied in Revision 4055. I've fixed the duplicate entries bugs. It was not due to the patch. :)
Be a part of the solution, not a part of the problem.