Need help with a component trying to use Lookup

Ok,

I'm using Absolute DB (componentace.com) with a table that is setup in the following way:

------------------------------------------------------|
BLink | OLink | Option                       | PBCode|
------------------------------------------------------|
201   | 1001  | 'Integral Dust Drawer' | 'IDD'    |
------------------------------------------------------|

So, this is a price book table, with BLink(book link), OLink(option link) and a PBCode which is a string value, and option text.  

I have a combo box component that I'm trying to automatically fill with option text based on an array that is passed to it.  Here's the code that I build the combo box with:

Var
  modelArray: Array [0..2] of String;
Begin

    modelArray[0] := 'GSMBASE';
    modelArray[1] := 'HOPPER';
    modelArray[2] := 'IDD';

    //_Configuration is the combo box I'm trying to build....
    _Configuration.LoadItemsFromCodes(modelArray);

//Here's the code for the combo box.  It has properties of BLink and OLink, which are integer values.  So in this case the BLink is set to 201:

Function TPBxComboBox.LoadItemsFromCodes(codeArray: Array Of String): Integer;
Var
    i,daOLink: Integer;
     SearchList, SearchList2: Variant;
Begin
    Result := -1; // error code for NIL value for datasets
    ErrMsg := 'Nil Datasets';
    If Not Assigned(dsOptions) Then
        Exit;
    If Not Assigned(PItem) Then
        Exit;

    xOLinkTrack := False;

    For i := 0 To High(codeArray) Do
    Begin
        With dsOptions As TABSTable Do
        Begin
           
            If State = dsInactive Then
                Open;

            Self.Items.Clear;
            xOLinkList.Clear; //this is a string list that contains an olink for each item in the combo box

            //lookup the OLink for the PBCode that was passed via the codeArray[] array:
            daOLink := dsOptions.Lookup('BLink;PBCode', VarArrayOf([BLink,codeArray[i]]), 'OLink');

            Self.xOLinkList.Add(IntToStr(daOLink));
           
            //use the Olink we just found to lookup the Option text for placing in the combo box
            Items.Add(dsOptions.Lookup('BLink;OLink', VarArrayOf([BLink, daOLink]), 'Option'));

            Self.ItemIndex := 0;
            Result := Items.Count;

            //other various code....
            If Items.Count > 0 Then
            Begin
                xOLinkTrack := True;
                ziHold      := ihOptions;
                Self.Change;
            End;
        End;
    End;
End;

As you can see, this should find an olink for the each of the values that I pass to the LoadItemsFromCodes function, then use that olink to find the option text and load that text into the combo box.  However, it only puts the 'IDD' or Integral Dust Drawer option in.  The two other array values are simply not present in the combo box, even though they are present in the dsOptions table and should be in the combo box.  I cannot figure out why.  This is my first time using the Lookup method, so I may be missing something easy here.

Thanks!
LVL 3
light_bulbAsked:
Who is Participating?
 
Wim ten BrinkConnect With a Mentor Self-employed developerCommented:
Damn! I was about to say that too. :-)

You might want to put the line "If State = dsInactive Then Open;" also a bit higher, outside the loop. No need to check if it's inactive for every item in the array, is it? This will probably speed up your code with about 5 clock cycles, which on a 1 GHz computer will save you... Well, it's not much, I know...

Go ask for a refund and have this question deleted before someone else sees your mistake... ;-)

0
 
light_bulbAuthor Commented:
Oh shoot, I found the problem.  I should have put the following lines before the loop instead of inside it:

Self.Items.Clear;
xOLinkList.Clear;

I was clearing the lists every time through the loop.  Oh well, not the first time I've made that mistake.
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.