RE: suggestion for taglib process

2004-01-05 Thread Edson Alves Pereira 
I´m not sure this is the best change for Generator, maybe i must
know tomcat´s source in deep. But is here what we need.

///
//Old
org.apache.tomcat.jasper.compiler.Generator:1450

String attrName = attrs[i].getName();
Method m = handlerInfo.getSetterMethod(attrName);
if (m == null) {
err.jspError(n, jsp.error.unable.to_find_method,
attrName);
}

///
New:
org.apache.tomcat.jasper.compiler.Generator:1450

StringBuffer sbMsg = null;
String attrName = attrs[i].getName();
Method m = handlerInfo.getSetterMethod(attrName);
if (m == null) {
sbMsg = new StringBuffer( );
sbMsg.append( jsp.error in [  );
sbMsg.append(
handlerInfo.getTagHandlerClass( ).getName( ));
sbMsg.append(  ], unable.to_find_method 
);

err.jspError(
n,
sbMsg.toString( ),
attrName);
}


 --
 De:   Shapira, Yoav[SMTP:[EMAIL PROTECTED]
 Responder:Tomcat Users List
 Enviada:  quarta-feira, 31 de dezembro de 2003 10:46
 Para: Tomcat Users List
 Assunto:  RE: suggestion for taglib process
 
 
 Howdy,
 Great!  I'm glad you have an enhancement in mind: when you contribute a
 patch I'll be glad to review and commit it. ;)
 
 Yoav Shapira
 Millennium ChemInformatics
 
 
 -Original Message-
 From: Edson Alves Pereira [mailto:[EMAIL PROTECTED]
 Sent: Tuesday, December 30, 2003 3:46 PM
 To: 'Tomcat-User List'
 Subject: suggestion for taglib process
 
   Hello folks, i´d like to see all error Tomcat´s messages related with
 taglib changed to display exactly the taglib that the error cames from,
 for
 example, take a look in this message:
 
 org.apache.jasper.JasperException:
 /jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter
 method
 for attribute: required
 
   I think it could be better if this message above would be like:
 
 org.apache.jasper.JasperException:
 /jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter
 method
 for attribute: required for oslib.tagext.NumberTag
 
   Regards,
   Edson
 
 
 
 
 This e-mail, including any attachments, is a confidential business
 communication, and may contain information that is confidential,
 proprietary and/or privileged.  This e-mail is intended only for the
 individual(s) to whom it is addressed, and may not be saved, copied,
 printed, disclosed or used by anyone else.  If you are not the(an)
 intended recipient, please immediately delete this e-mail from your
 computer system and notify the sender.  Thank you.
 
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]

RE: suggestion for taglib process

2004-01-05 Thread Shapira, Yoav 

Howdy,
Actually, this seems very reasonable.  Please submit it to Bugzilla as an enhancement 
request and attach your code diff.  Thanks ;)

Yoav Shapira
Millennium ChemInformatics


-Original Message-
From: Edson Alves Pereira [mailto:[EMAIL PROTECTED]
Sent: Monday, January 05, 2004 9:46 AM
To: 'Tomcat Users List'
Subject: RE: suggestion for taglib process

   I´m not sure this is the best change for Generator, maybe i must
know tomcat´s source in deep. But is here what we need.

///
//Old
org.apache.tomcat.jasper.compiler.Generator:1450

   String attrName = attrs[i].getName();
   Method m = handlerInfo.getSetterMethod(attrName);
   if (m == null) {
   err.jspError(n, jsp.error.unable.to_find_method,
   attrName);
   }

///
New:
org.apache.tomcat.jasper.compiler.Generator:1450

   StringBuffer sbMsg = null;
   String attrName = attrs[i].getName();
   Method m = handlerInfo.getSetterMethod(attrName);
   if (m == null) {
   sbMsg = new StringBuffer( );
   sbMsg.append( jsp.error in [  );
   sbMsg.append(
handlerInfo.getTagHandlerClass( ).getName( ));
   sbMsg.append(  ], unable.to_find_method 
);

   err.jspError(
   n,
   sbMsg.toString( ),
   attrName);
   }


 --
 De:  Shapira, Yoav[SMTP:[EMAIL PROTECTED]
 Responder:   Tomcat Users List
 Enviada: quarta-feira, 31 de dezembro de 2003 10:46
 Para:Tomcat Users List
 Assunto: RE: suggestion for taglib process


 Howdy,
 Great!  I'm glad you have an enhancement in mind: when you contribute a
 patch I'll be glad to review and commit it. ;)

 Yoav Shapira
 Millennium ChemInformatics


 -Original Message-
 From: Edson Alves Pereira [mailto:[EMAIL PROTECTED]
 Sent: Tuesday, December 30, 2003 3:46 PM
 To: 'Tomcat-User List'
 Subject: suggestion for taglib process
 
   Hello folks, i´d like to see all error Tomcat´s messages related with
 taglib changed to display exactly the taglib that the error cames from,
 for
 example, take a look in this message:
 
 org.apache.jasper.JasperException:
 /jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter
 method
 for attribute: required
 
   I think it could be better if this message above would be like:
 
 org.apache.jasper.JasperException:
 /jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter
 method
 for attribute: required for oslib.tagext.NumberTag
 
   Regards,
   Edson




 This e-mail, including any attachments, is a confidential business
 communication, and may contain information that is confidential,
 proprietary and/or privileged.  This e-mail is intended only for the
 individual(s) to whom it is addressed, and may not be saved, copied,
 printed, disclosed or used by anyone else.  If you are not the(an)
 intended recipient, please immediately delete this e-mail from your
 computer system and notify the sender.  Thank you.


 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]




This e-mail, including any attachments, is a confidential business communication, and 
may contain information that is confidential, proprietary and/or privileged.  This 
e-mail is intended only for the individual(s) to whom it is addressed, and may not be 
saved, copied, printed, disclosed or used by anyone else.  If you are not the(an) 
intended recipient, please immediately delete this e-mail from your computer system 
and notify the sender.  Thank you.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

RE: suggestion for taglib process

2004-01-05 Thread Edson Alves Pereira 
Hello Yoav, i think we have here more that just changing a error
message. I´ll try to explain:

The class that jasper says not found a method is named
CurrencyLookup and its method is setRequired, that´s why jasper complains
about something called required ( i telling you what is happening ).
CurrencyLookup extends Lookup and Lookup extends FieldHTML ( all of them are
taglibs ). setRequired is implemented in FieldHTML. After debug tomcat and
check which class jasper was complaining, i really couldn´t understand why
Generator can´t see this method. For me its sound like another bug, what do
you think.

PS. I´m using tomcat-4.1.24


 --
 De:   Shapira, Yoav[SMTP:[EMAIL PROTECTED]
 Responder:Tomcat Users List
 Enviada:  segunda-feira, 5 de janeiro de 2004 11:19
 Para: Tomcat Users List
 Assunto:  RE: suggestion for taglib process
 
 
 Howdy,
 Actually, this seems very reasonable.  Please submit it to Bugzilla as an
 enhancement request and attach your code diff.  Thanks ;)
 
 Yoav Shapira
 Millennium ChemInformatics
 
 
 -Original Message-
 From: Edson Alves Pereira [mailto:[EMAIL PROTECTED]
 Sent: Monday, January 05, 2004 9:46 AM
 To: 'Tomcat Users List'
 Subject: RE: suggestion for taglib process
 
  I´m not sure this is the best change for Generator, maybe i must
 know tomcat´s source in deep. But is here what we need.
 
 /
 //
 //Old
 org.apache.tomcat.jasper.compiler.Generator:1450
 
  String attrName = attrs[i].getName();
  Method m = handlerInfo.getSetterMethod(attrName);
  if (m == null) {
  err.jspError(n, jsp.error.unable.to_find_method,
  attrName);
  }
 
 /
 //
 New:
 org.apache.tomcat.jasper.compiler.Generator:1450
 
  StringBuffer sbMsg = null;
  String attrName = attrs[i].getName();
  Method m = handlerInfo.getSetterMethod(attrName);
  if (m == null) {
  sbMsg = new StringBuffer( );
  sbMsg.append( jsp.error in [  );
  sbMsg.append(
 handlerInfo.getTagHandlerClass( ).getName( ));
  sbMsg.append(  ], unable.to_find_method 
 );
 
  err.jspError(
  n,
  sbMsg.toString( ),
  attrName);
  }
 
 
  --
  De:Shapira, Yoav[SMTP:[EMAIL PROTECTED]
  Responder: Tomcat Users List
  Enviada:   quarta-feira, 31 de dezembro de 2003 10:46
  Para:  Tomcat Users List
  Assunto:   RE: suggestion for taglib process
 
 
  Howdy,
  Great!  I'm glad you have an enhancement in mind: when you contribute a
  patch I'll be glad to review and commit it. ;)
 
  Yoav Shapira
  Millennium ChemInformatics
 
 
  -Original Message-
  From: Edson Alves Pereira [mailto:[EMAIL PROTECTED]
  Sent: Tuesday, December 30, 2003 3:46 PM
  To: 'Tomcat-User List'
  Subject: suggestion for taglib process
  
Hello folks, i´d like to see all error Tomcat´s messages related
 with
  taglib changed to display exactly the taglib that the error cames
 from,
  for
  example, take a look in this message:
  
  org.apache.jasper.JasperException:
  /jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter
  method
  for attribute: required
  
I think it could be better if this message above would be like:
  
  org.apache.jasper.JasperException:
  /jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter
  method
  for attribute: required for oslib.tagext.NumberTag
  
Regards,
Edson
 
 
 
 
  This e-mail, including any attachments, is a confidential business
  communication, and may contain information that is confidential,
  proprietary and/or privileged.  This e-mail is intended only for the
  individual(s) to whom it is addressed, and may not be saved, copied,
  printed, disclosed or used by anyone else.  If you are not the(an)
  intended recipient, please immediately delete this e-mail from your
  computer system and notify the sender.  Thank you.
 
 
  -
  To unsubscribe, e-mail: [EMAIL PROTECTED]
  For additional commands, e-mail: [EMAIL PROTECTED]
 
 
 
 
 This e-mail, including any attachments, is a confidential business
 communication, and may contain information that is confidential,
 proprietary and/or privileged.  This e-mail is intended only for the
 individual(s) to whom it is addressed, and may not be saved, copied,
 printed, disclosed or used by anyone else.  If you are not the(an)
 intended recipient, please immediately delete this e-mail from your
 computer system and notify the sender.  Thank you.
 
 
 -
 To unsubscribe

RE: suggestion for taglib process

2003-12-31 Thread Shapira, Yoav 

Howdy,
Great!  I'm glad you have an enhancement in mind: when you contribute a patch I'll be 
glad to review and commit it. ;)

Yoav Shapira
Millennium ChemInformatics


-Original Message-
From: Edson Alves Pereira [mailto:[EMAIL PROTECTED]
Sent: Tuesday, December 30, 2003 3:46 PM
To: 'Tomcat-User List'
Subject: suggestion for taglib process

  Hello folks, i´d like to see all error Tomcat´s messages related with
taglib changed to display exactly the taglib that the error cames from, for
example, take a look in this message:

org.apache.jasper.JasperException:
/jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter method
for attribute: required

  I think it could be better if this message above would be like:

org.apache.jasper.JasperException:
/jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter method
for attribute: required for oslib.tagext.NumberTag

  Regards,
  Edson




This e-mail, including any attachments, is a confidential business communication, and 
may contain information that is confidential, proprietary and/or privileged.  This 
e-mail is intended only for the individual(s) to whom it is addressed, and may not be 
saved, copied, printed, disclosed or used by anyone else.  If you are not the(an) 
intended recipient, please immediately delete this e-mail from your computer system 
and notify the sender.  Thank you.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

suggestion for taglib process

2003-12-30 Thread Edson Alves Pereira 
  Hello folks, i´d like to see all error Tomcat´s messages related with
taglib changed to display exactly the taglib that the error cames from, for
example, take a look in this message:

org.apache.jasper.JasperException:
/jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter method
for attribute: required

  I think it could be better if this message above would be like:

org.apache.jasper.JasperException:
/jsp/cadastro/OS_cadastroProjetoEd.jsp(101,6) Unable to find setter method
for attribute: required for oslib.tagext.NumberTag

  Regards,
  Edson