You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by victorsosa <gi...@git.apache.org> on 2016/01/07 16:21:42 UTC

[GitHub] struts pull request: patch ww-3266 getFieldValue() method of Value...

GitHub user victorsosa opened a pull request:

    https://github.com/apache/struts/pull/71

    patch ww-3266 getFieldValue() method of ValueStackDataSource uses getDescription() to get field value may not appropriate.

    patch ww-3266
    
    getFieldValue() method of ValueStackDataSource uses getDescription() to get field value may not appropriate.
    
    Reason:
    In JasperReports, a description of a field is just like a documentation for this field. So it may not good to use it as expression

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/victorsosa/struts WW-3266

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/struts/pull/71.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #71
    
----
commit 7170c13509611284b100521968163698e96708b9
Author: victor sosa <vi...@users.noreply.github.com>
Date:   2016-01-04T22:06:23Z

    Merge pull request #1 from apache/master
    
    Merge changes

commit 37f822f9cce8505e28a3dba5a2ac34aa03a17dd2
Author: Victor Sosa <vi...@gmail.com>
Date:   2016-01-07T15:09:58Z

    Patch WW-3266
    
    getFieldValue() method of ValueStackDataSource uses getDescription() to
    get field value may not appropriate

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by lukaszlenart <gi...@git.apache.org>.
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/71
  
    Please open an issue in JIRA with all these details


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by DenizR <gi...@git.apache.org>.
Github user DenizR commented on the issue:

    https://github.com/apache/struts/pull/71
  
    I have tested all old and latest version of all struts2 core and struts2 jasper reports plugin version without success if I call in my report A a subreport B via "`new net.sf.jasperreports.engine.data.JRBeanCollectionDataSource($F{maintenanceBookletItemDAOList})"` and 
    "`<result name="success" **type="jasper"** >`"
    ```
    <action name="printAllMaintenanceBookletEntries" class="..myhiddenpackage..MaintenanceBookletAction" method="printAllMaintenanceBookletEntries">
    		<result name="success" type="jasper" >
    			<param name="location">/jasper/MaintenanceBookletEntries.jasper</param>
    			<param name="dataSource">allMaintenanceBookletPrintDAOList</param>
    			<param name="format">PDF</param>
    		</result>
    		<result name="error">
    			/jsp/modernLayout/maintenanceBooklet/maintenanceEntries.jsp
    		</result>
    	</action> 
    ```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts pull request: patch ww-3266 getFieldValue() method of Value...

Posted by victorsosa <gi...@git.apache.org>.
Github user victorsosa commented on the pull request:

    https://github.com/apache/struts/pull/71#issuecomment-172194026
  
    :+1: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by DenizR <gi...@git.apache.org>.
Github user DenizR commented on the issue:

    https://github.com/apache/struts/pull/71
  
    @lukaszlenart: Thank you very much for your response.  I am not sure if this is the same issue regarding ValueStackDataSource.  I get the following error if I call in my jasper report \u201cA\u201d  a subreport \u201cB\u201d. 
    "Caused by: java.lang.ClassCastException: org.apache.struts2.views.jasperreports.ValueStackDataSource cannot be cast to java.util.Collection" 
    This errors occurs upwards struts2 version 2.3.16. Same code will be executed without errors until struts2 core 2.3.15.3 and struts 2 Jasper Reports Plugin version 2.3.15.3. My defined variable \u201c$F{maintenanceBookletItemDAOList})\u201d is a java.util.collection.
    
    net.sf.jasperreports.engine.fill.JRExpressionEvalException: Error evaluating expression : 
    	Source text : new net.sf.jasperreports.engine.data.JRBeanCollectionDataSource($F{maintenanceBookletItemDAOList})
    	at net.sf.jasperreports.engine.fill.JREvaluator.evaluate(JREvaluator.java:203)
    	at net.sf.jasperreports.engine.fill.JRCalculator.evaluate(JRCalculator.java:591)
    	at net.sf.jasperreports.engine.fill.JRCalculator.evaluate(JRCalculator.java:559)
    	at net.sf.jasperreports.engine.fill.JRFillElement.evaluateExpression(JRFillElement.java:876)
    	at net.sf.jasperreports.engine.fill.JRFillSubreport.evaluateSubreport(JRFillSubreport.java:356)
    	at net.sf.jasperreports.engine.fill.JRFillSubreport.evaluate(JRFillSubreport.java:275)
    	at net.sf.jasperreports.engine.fill.JRFillElementContainer.evaluate(JRFillElementContainer.java:257)
    	at net.sf.jasperreports.engine.fill.JRFillBand.evaluate(JRFillBand.java:473)
    	at net.sf.jasperreports.engine.fill.JRVerticalFiller.fillColumnBand(JRVerticalFiller.java:2021)
    	at net.sf.jasperreports.engine.fill.JRVerticalFiller.fillDetail(JRVerticalFiller.java:755)
    	at net.sf.jasperreports.engine.fill.JRVerticalFiller.fillReportStart(JRVerticalFiller.java:265)
    	at net.sf.jasperreports.engine.fill.JRVerticalFiller.fillReport(JRVerticalFiller.java:128)
    	at net.sf.jasperreports.engine.fill.JRBaseFiller.fill(JRBaseFiller.java:836)
    	at net.sf.jasperreports.engine.fill.JRBaseFiller.fill(JRBaseFiller.java:765)
    	at net.sf.jasperreports.engine.fill.JRFiller.fillReport(JRFiller.java:84)
    	at net.sf.jasperreports.engine.JasperFillManager.fillReport(JasperFillManager.java:624)
    	at org.apache.struts2.views.jasperreports.JasperReportsResult.doExecute(JasperReportsResult.java:325)
    	at org.apache.struts2.dispatcher.StrutsResultSupport.execute(StrutsResultSupport.java:186)
    	at com.opensymphony.xwork2.DefaultActionInvocation.executeResult(DefaultActionInvocation.java:371)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:275)
    	at org.apache.struts2.interceptor.DeprecationInterceptor.intercept(DeprecationInterceptor.java:41)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at org.apache.struts2.interceptor.debugging.DebuggingInterceptor.intercept(DebuggingInterceptor.java:256)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.DefaultWorkflowInterceptor.doIntercept(DefaultWorkflowInterceptor.java:167)
    	at com.opensymphony.xwork2.interceptor.MethodFilterInterceptor.intercept(MethodFilterInterceptor.java:98)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.validator.ValidationInterceptor.doIntercept(ValidationInterceptor.java:265)
    	at org.apache.struts2.interceptor.validation.AnnotationValidationInterceptor.doIntercept(AnnotationValidationInterceptor.java:68)
    	at com.opensymphony.xwork2.interceptor.MethodFilterInterceptor.intercept(MethodFilterInterceptor.java:98)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ConversionErrorInterceptor.intercept(ConversionErrorInterceptor.java:138)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ParametersInterceptor.doIntercept(ParametersInterceptor.java:249)
    	at com.opensymphony.xwork2.interceptor.MethodFilterInterceptor.intercept(MethodFilterInterceptor.java:98)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ParametersInterceptor.doIntercept(ParametersInterceptor.java:249)
    	at com.opensymphony.xwork2.interceptor.MethodFilterInterceptor.intercept(MethodFilterInterceptor.java:98)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.StaticParametersInterceptor.intercept(StaticParametersInterceptor.java:191)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at org.apache.struts2.interceptor.MultiselectInterceptor.intercept(MultiselectInterceptor.java:73)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at org.apache.struts2.interceptor.CheckboxInterceptor.intercept(CheckboxInterceptor.java:91)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at org.apache.struts2.interceptor.FileUploadInterceptor.intercept(FileUploadInterceptor.java:252)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ModelDrivenInterceptor.intercept(ModelDrivenInterceptor.java:100)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ScopedModelDrivenInterceptor.intercept(ScopedModelDrivenInterceptor.java:141)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ChainingInterceptor.intercept(ChainingInterceptor.java:145)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.PrepareInterceptor.doIntercept(PrepareInterceptor.java:171)
    	at com.opensymphony.xwork2.interceptor.MethodFilterInterceptor.intercept(MethodFilterInterceptor.java:98)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.I18nInterceptor.intercept(I18nInterceptor.java:139)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at org.apache.struts2.interceptor.ServletConfigInterceptor.intercept(ServletConfigInterceptor.java:164)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.AliasInterceptor.intercept(AliasInterceptor.java:193)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at com.opensymphony.xwork2.interceptor.ExceptionMappingInterceptor.intercept(ExceptionMappingInterceptor.java:189)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at eu.mtd.interceptors.MTDUserDefinedLayoutSwitchInterceptor.intercept(MTDUserDefinedLayoutSwitchInterceptor.java:60)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at eu.mtd.interceptors.LoginCheckInterceptor.intercept(LoginCheckInterceptor.java:60)
    	at com.opensymphony.xwork2.DefaultActionInvocation.invoke(DefaultActionInvocation.java:246)
    	at org.apache.struts2.impl.StrutsActionProxy.execute(StrutsActionProxy.java:54)
    	at org.apache.struts2.dispatcher.Dispatcher.serviceAction(Dispatcher.java:562)
    	at org.apache.struts2.dispatcher.ng.ExecuteOperations.executeAction(ExecuteOperations.java:77)
    	at org.apache.struts2.dispatcher.ng.filter.StrutsPrepareAndExecuteFilter.doFilter(StrutsPrepareAndExecuteFilter.java:99)
    	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
    	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
    	at org.springframework.orm.jpa.support.OpenEntityManagerInViewFilter.doFilterInternal(OpenEntityManagerInViewFilter.java:178)
    	at org.springframework.web.filter.OncePerRequestFilter.doFilter(OncePerRequestFilter.java:107)
    	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
    	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
    	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:233)
    	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:191)
    	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:127)
    	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:103)
    	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
    	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:293)
    	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:861)
    	at org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:606)
    	at org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:489)
    	at java.lang.Thread.run(Unknown Source)
    Caused by: java.lang.ClassCastException: org.apache.struts2.views.jasperreports.ValueStackDataSource cannot be cast to java.util.Collection
    	at MaintenanceBookletEntry_1493377131470_196207.evaluate(MaintenanceBookletEntry_1493377131470_196207:517)
    	at net.sf.jasperreports.engine.fill.JREvaluator.evaluate(JREvaluator.java:190)
    	... 87 more



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by lukaszlenart <gi...@git.apache.org>.
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/71
  
    As far I can recall, there were some changes but please try to use 2.3.20 or even the latest version 2.3.32


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by DenizR <gi...@git.apache.org>.
Github user DenizR commented on the issue:

    https://github.com/apache/struts/pull/71
  
    Which website is that please? There are many portals of JIRA. Thanks


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by lukaszlenart <gi...@git.apache.org>.
Github user lukaszlenart commented on the issue:

    https://github.com/apache/struts/pull/71
  
    @DenizR I'm not sure what do you mean by that, but the mentioned issue was fixed in Struts 2.3.20


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts pull request: patch ww-3266 getFieldValue() method of Value...

Posted by lukaszlenart <gi...@git.apache.org>.
Github user lukaszlenart commented on the pull request:

    https://github.com/apache/struts/pull/71#issuecomment-172174042
  
    Merged, thanks!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by DenizR <gi...@git.apache.org>.
Github user DenizR commented on the issue:

    https://github.com/apache/struts/pull/71
  
    This bug orrurs after the struts2 version 2.3.15.3 (Oct 2013) although it will be stated that the error has been fixed  https://issues.apache.org/jira/browse/WW-3698 
    When can one expect a solution?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


[GitHub] struts issue #71: patch ww-3266 getFieldValue() method of ValueStackDataSour...

Posted by aleksandr-m <gi...@git.apache.org>.
Github user aleksandr-m commented on the issue:

    https://github.com/apache/struts/pull/71
  
    @DenizR https://issues.apache.org/jira/browse/WW/


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org