You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Tam Du (JIRA)" <ji...@apache.org> on 2009/05/14 12:31:45 UTC

[jira] Created: (TAP5-697) Tapestry_improment_ideas

Tapestry_improment_ideas
------------------------

                 Key: TAP5-697
                 URL: https://issues.apache.org/jira/browse/TAP5-697
             Project: Tapestry 5
          Issue Type: Improvement
          Components: tapestry-core
    Affects Versions: 5.0.18, 5.1.0.5
            Reporter: Tam Du


During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.

3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
For example:
    I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
             configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
    I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
             configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");

4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
        <filter>
              <filter-name>app</filter-name>
              <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
              <ignore-pattern>/messageBroker/</ignore-pattern> 
        </filter>

5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
     Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
     Solution 2:   Save tml file in the WEB_INF folder of project.

I hope that Tapestry will be improved these features. 
Thank you very much.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du updated TAP5-697:
------------------------

    Summary: Tapestry_improvement_ideas  (was: Tapestry_improment_ideas)

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>
> During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
> 1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.
> 2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
> For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.
> 3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
> For example:
>     I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
>     I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");
> 4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
> For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
>         <filter>
>               <filter-name>app</filter-name>
>               <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
>               <ignore-pattern>/messageBroker/</ignore-pattern> 
>         </filter>
> 5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
>      Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
>      Solution 2:   Save tml file in the WEB_INF folder of project.
> I hope that Tapestry will be improved these features. 
> Thank you very much.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12710158#action_12710158 ] 

Tam Du commented on TAP5-697:
-----------------------------

Thanks a lot for your consideration.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>            Assignee: Robert Zeigler
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du reopened TAP5-697:
-------------------------


I know that OutputRaw component can do that. However, I want to explain my idea.

At first, please consider my example:
I have the string "<Hello>".  I store it in the database. My software will filter this string like this "&lt;Hello&gt;" and store it in database.
In .tml file, I have 2 variables ${hello1} and ${hello2}. 
In java file, I have 2 methods 
      getHello1() {...} returns my original string "<Hello>".
      getHello2() {...} returns my stored String in database. Therefore, it returns "&lt;Hello&gt;".
I hope that when running this example, it will show string "&lt;Hello&gt;" 2 times.
But the result is 2 different strings: "&lt;Hello&gt;" and "&amp;lt;Hello&amp;gt;".

I agree that it should be always filter the HTML data. But it also should check if the HTML data was already filtered. If the data have been filtered already, Tapestry should not filter it anymore.


> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (TAP5-697) Tapestry_improvement_ideas

Posted by "Robert Zeigler (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Zeigler closed TAP5-697.
-------------------------------

    Assignee: Robert Zeigler

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>            Assignee: Robert Zeigler
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du updated TAP5-697:
------------------------

    Attachment: DisableHtmlTag.java

class is used to prevent disable html the second time.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
> 1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.
> 2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
> For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.
> 3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
> For example:
>     I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
>     I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");
> 4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
> For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
>         <filter>
>               <filter-name>app</filter-name>
>               <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
>               <ignore-pattern>/messageBroker/</ignore-pattern> 
>         </filter>
> 5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
>      Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
>      Solution 2:   Save tml file in the WEB_INF folder of project.
> I hope that Tapestry will be improved these features. 
> Thank you very much.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TAP5-697) Tapestry_improvement_ideas

Posted by "Filip S. Adamsen (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709816#action_12709816 ] 

Filip S. Adamsen commented on TAP5-697:
---------------------------------------

Problem is, there's no way for Tapestry to *know* that the data has already been filtered - if &amp;lt;Hello&amp;gt; was the actual, unfiltered data, this wouldn't work.

Anyway, just follow Thiago's advice and ust OutputRaw or create a "raw" binding prefix if it's something you'll need to do a lot. You could even make an alwaysraw binding prefix to do what you want.

But it's not something that should be changed in Tapestry - that's my €0.02 anyway.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du updated TAP5-697:
------------------------

    Description: In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.  (was: During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.

3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
For example:
    I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
             configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
    I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
             configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");

4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
        <filter>
              <filter-name>app</filter-name>
              <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
              <ignore-pattern>/messageBroker/</ignore-pattern> 
        </filter>

5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
     Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
     Solution 2:   Save tml file in the WEB_INF folder of project.

I hope that Tapestry will be improved these features. 
Thank you very much.
)

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du updated TAP5-697:
------------------------

    Summary: Tapestry_improvement_ideas  (was: Tapestry_improment_ideas)

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>
> During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
> 1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.
> 2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
> For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.
> 3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
> For example:
>     I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
>     I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");
> 4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
> For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
>         <filter>
>               <filter-name>app</filter-name>
>               <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
>               <ignore-pattern>/messageBroker/</ignore-pattern> 
>         </filter>
> 5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
>      Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
>      Solution 2:   Save tml file in the WEB_INF folder of project.
> I hope that Tapestry will be improved these features. 
> Thank you very much.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TAP5-697) Tapestry_improvement_ideas

Posted by "Filip S. Adamsen (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709816#action_12709816 ] 

Filip S. Adamsen commented on TAP5-697:
---------------------------------------

Problem is, there's no way for Tapestry to *know* that the data has already been filtered - if &amp;lt;Hello&amp;gt; was the actual, unfiltered data, this wouldn't work.

Anyway, just follow Thiago's advice and ust OutputRaw or create a "raw" binding prefix if it's something you'll need to do a lot. You could even make an alwaysraw binding prefix to do what you want.

But it's not something that should be changed in Tapestry - that's my €0.02 anyway.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (TAP5-697) Tapestry_improvement_ideas

Posted by "Robert Zeigler (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Zeigler closed TAP5-697.
-------------------------------

    Assignee: Robert Zeigler

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>            Assignee: Robert Zeigler
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (TAP5-697) Tapestry_improvement_ideas

Posted by "Thiago H. de Paula Figueiredo (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Thiago H. de Paula Figueiredo closed TAP5-697.
----------------------------------------------

    Resolution: Invalid

There is a builtin Tapestry component, OutputRaw, that does exactly what you need: http://tapestry.apache.org/tapestry5.1/tapestry-core/ref/org/apache/tapestry5/corelib/components/OutputRaw.html It outputs HTML without converting < into &lt;, for example.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TAP5-697) Tapestry_improvement_ideas

Posted by "Robert Zeigler (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709819#action_12709819 ] 

Robert Zeigler commented on TAP5-697:
-------------------------------------

As pointed out, this is the purpose of the OutputRaw component. And as Filip said, there's no way for tapestry to know whether the data has been filtered or not.   This is knowledge you must explicitly declare as a user.  And the way to declare that knowledge is to use the OutputRaw component.  Or, as Filip suggested, look into creating a "raw" binding prefix.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (TAP5-697) Tapestry_improvement_ideas

Posted by "Robert Zeigler (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Zeigler resolved TAP5-697.
---------------------------------

    Resolution: Invalid

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du updated TAP5-697:
------------------------

    Attachment: DisableHtmlTag.java

class is used to prevent disable html the second time.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
> 1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.
> 2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
> For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.
> 3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
> For example:
>     I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
>     I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
>              configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");
> 4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
> For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
>         <filter>
>               <filter-name>app</filter-name>
>               <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
>               <ignore-pattern>/messageBroker/</ignore-pattern> 
>         </filter>
> 5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
>      Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
>      Solution 2:   Save tml file in the WEB_INF folder of project.
> I hope that Tapestry will be improved these features. 
> Thank you very much.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du updated TAP5-697:
------------------------

    Description: In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.  (was: During the time using Tapestry, I found some feature can be improved for Tapestry. Here they are:
1.	In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

2.	When using Grid component of Tapestry, I found that we should have an option to display/not display the header of the Grid. The reason of this requirement is: when developing my product, in some cases, I need to use Grid but I don't want to display the header of that grid. 
For example, I use Tapestry to write the list topic page http://62.101.68.227:8888/snippetrepobrowser/, the header of Grid is unnecessary.

3.	In the current version of Tapestry framework, I can only configure my project in only one ".properties" file. This is no problem with a simple project but with a complex project which requires other projects. In complex projects, I must copy/paste keys in properties file from one project to another. It makes me hard to maintain and manage my projects as well as my products. So, I hope that Tapestry can support multi properties file in each project (especially in the complex projects).
For example:
    I have this code to set the value of key APPLICATION_ CATALOG in file í8n.properties
             configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties"); 
    I would like to set the value of key APPLICATION_CATALOG in other file properties, like this:
             configuration.add (SymbolConstants.APPLICATION_CATALOG, "WEB-INF/classes/i18n/i18n.properties; WEB-INF/classes/i18n/i18n_vn.properties;");

4.	 I use Tapestry and Flex Blaze-DS to develop my project. Current TapestryFilter class filters not only Tapestry patterns but also Flex Blaze-DS patterns. I think it had better have an option so that we can control what Tapestry filters.  In this case, I suggest that we need to improve TapestryFilter class for the purpose of ignoring any patterns that developers don't want Tapestry to filter.
For example: I want TapestryFilter ignore some patterns which contain string "/messageBroker/". The source code may be like this:
        <filter>
              <filter-name>app</filter-name>
              <filter-class>org.apache.tapestry5.TapestryFilter</filter-class>
              <ignore-pattern>/messageBroker/</ignore-pattern> 
        </filter>

5.	I request this feature for purpose of the security of website. Currently, I found that users can download tml files by putting ".tml" at the end of URL. Users can do like that because .tml files are put in the webroot folder of the project. To prevent download, I have 2 solutions for this case:
     Solution 1:   .tml files can be saved anywhere in project, and Tapestry will support the configuration to determine where to save these files.
     Solution 2:   Save tml file in the WEB_INF folder of project.

I hope that Tapestry will be improved these features. 
Thank you very much.
)

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (TAP5-697) Tapestry_improvement_ideas

Posted by "Robert Zeigler (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robert Zeigler resolved TAP5-697.
---------------------------------

    Resolution: Invalid

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12710158#action_12710158 ] 

Tam Du commented on TAP5-697:
-----------------------------

Thanks a lot for your consideration.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>            Assignee: Robert Zeigler
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Closed: (TAP5-697) Tapestry_improvement_ideas

Posted by "Thiago H. de Paula Figueiredo (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Thiago H. de Paula Figueiredo closed TAP5-697.
----------------------------------------------

    Resolution: Invalid

There is a builtin Tapestry component, OutputRaw, that does exactly what you need: http://tapestry.apache.org/tapestry5.1/tapestry-core/ref/org/apache/tapestry5/corelib/components/OutputRaw.html It outputs HTML without converting < into &lt;, for example.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Reopened: (TAP5-697) Tapestry_improvement_ideas

Posted by "Tam Du (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tam Du reopened TAP5-697:
-------------------------


I know that OutputRaw component can do that. However, I want to explain my idea.

At first, please consider my example:
I have the string "<Hello>".  I store it in the database. My software will filter this string like this "&lt;Hello&gt;" and store it in database.
In .tml file, I have 2 variables ${hello1} and ${hello2}. 
In java file, I have 2 methods 
      getHello1() {...} returns my original string "<Hello>".
      getHello2() {...} returns my stored String in database. Therefore, it returns "&lt;Hello&gt;".
I hope that when running this example, it will show string "&lt;Hello&gt;" 2 times.
But the result is 2 different strings: "&lt;Hello&gt;" and "&amp;lt;Hello&amp;gt;".

I agree that it should be always filter the HTML data. But it also should check if the HTML data was already filtered. If the data have been filtered already, Tapestry should not filter it anymore.


> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (TAP5-697) Tapestry_improvement_ideas

Posted by "Robert Zeigler (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TAP5-697?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12709819#action_12709819 ] 

Robert Zeigler commented on TAP5-697:
-------------------------------------

As pointed out, this is the purpose of the OutputRaw component. And as Filip said, there's no way for tapestry to know whether the data has been filtered or not.   This is knowledge you must explicitly declare as a user.  And the way to declare that knowledge is to use the OutputRaw component.  Or, as Filip suggested, look into creating a "raw" binding prefix.

> Tapestry_improvement_ideas
> --------------------------
>
>                 Key: TAP5-697
>                 URL: https://issues.apache.org/jira/browse/TAP5-697
>             Project: Tapestry 5
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5, 5.0.18
>            Reporter: Tam Du
>         Attachments: DisableHtmlTag.java
>
>
> In the tml file, we should not disable html the second time, that is, if it is already disabled, then should not disable again. For more details, I attach here my file "DisableHTMLTag.java" which I have been using to not disable html the second time. I hope that you can improve this feature in the next version.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.