🚀 go-pugleaf

Randy <slyfox65@netzero.net> wrote:
> it execute the entire dialog constructor, but nothing is shown.

Well, yes.  The JournalAction constructor will run when you first create
the instance, with the line:

        dateAction = new JournalAction("viewByDate",this);

This does not test the need for the menu item itself to properly trigger
the actionPerformed method on Journal_Ledger itself.  This is what I think
is failing.

Here is what I was suggesting you try changing:

public class JournalAction extends AbstractAction {
  private Journal_Ledger target;

  public JournalAction(String name,Journal_Ledger comp) {
       putValue(Action.NAME,name);
       target = comp;
  }

  public void actionPerformed(ActionEvent evnt) {
// ADD THE NEXT TWO LINES and the other prints, too.
    System.out.println("IN actionPerformed: " + evnt.toString());
    System.out.println("Action.NAME: " + Action.NAME);
      if(getValue(Action.NAME) == "viewByLedger") {
             target.test();
         System.out.println("target.test triggered");
      }
      else if(getValue(Action.NAME) == "viewByDate") {
        target.viewRecord(1);
        System.out.println("target.viewRecord triggered");
      }
    /**@todo: implement this javax.swing.AbstractAction abstract method*/

  }
}

If you invoke your menu item and you don't see the print statement, then
your menu item isn't listening to the correct thing.

If it does print, look carefully at the contents of the event object and
see if it contains what you expected it to contain.

Question:  in viewRecord() you have a jif.show(), and then you add it to
the JFrame's contentPane.  Might the contentPane not be invalidating at
this point?  I don't know if this is good or not, I still resort to trial
and error more than I care to when it comes to packing frames.  (This is
one place where Perl/tk beats out java...there's only one pack() trigger,
rather than pack(), validate(), invalidate(), revalidate()....!)

-- 
Joi Ellis (TeamB) 
No direct email, please.
For best results, post on news server newsgroups.borland.com!