Ignoring paths containing symbols works fine with me. I’d otherwise not have a 
problem with a project-based configuration file. Maybe it should live under 
WEB-INF or even META-INF.

Norman Franke
Answering Service for Directors, Inc.
www.myasd.com



On Feb 4, 2015, at 11:46 AM, Dmitry Gusev <dmitry.gu...@gmail.com> wrote:

> Hey Norman,
> 
> I'll come with some solution in the next release. Probably will just ignore
> validation for paths containing symbols.
> 
> Regarding static analysis of tapestry's symbols - it won't be easy to do,
> because the values for them calculated at runtime using method invocations
> and string manipulations... Probably time for some configuration file where
> one can specify missing bits & pieces.
> 
> I was thinking to store plugin configuration in the project's source code.
> Somewhere under 'src/main/tapestry/eclipse-plugin-config.json'. What do you
> think?
> 
> On Wed, Feb 4, 2015 at 7:29 PM, Norman Franke <nor...@myasd.com> wrote:
> 
>> Speaking of which, any chance the plugin could not warn about @Import
>> using "${tapestry.scriptaculous}/“ paths? Or is there a better way to
>> include things like effects.js and builder.js?
>> 
>> Norman Franke
>> Answering Service for Directors, Inc.
>> www.myasd.com
>> 
>> 
>> 
>> On Feb 4, 2015, at 2:15 AM, Dmitry Gusev <dmitry.gu...@gmail.com> wrote:
>> 
>>> Hi Martin,
>>> 
>>> I'm sorry I was unclear, the symbol expansion support is not yet in the
>>> plugin.
>>> I was just saying that it shouldn't be hard to add the support for simple
>>> symbol contributions :)
>>> 
>>> I'll try to add it in the next release.
>>> 
>>> On Wed, Feb 4, 2015 at 7:15 AM, Martin Papy <martin.p...@gmail.com>
>> wrote:
>>> 
>>>> Hi Dmitry,
>>>> 
>>>> My code was slightly different, but I updated it to see if there was any
>>>> difference. So now I have this :
>>>> 
>>>>   public static void
>>>> contributeApplicationDefaults(MappedConfiguration<String, String>
>>>> configuration) {
>>>>       configuration.add("layout.assets", "META-INF/assets");
>>>>       configuration.add("layout.scripts", "META-INF/assets/js");
>>>>       configuration.add("layout.styles", "META-INF/assets/css");
>>>>       configuration.add("layout.images", "META-INF/assets/images");
>>>>   }
>>>> 
>>>> But the Tapestry Context is still unable to resolve the following :
>>>> 
>>>> @Import(library={
>>>>   "${layout.scripts}/jquery.confirm.js"
>>>> }, module={
>>>>   "bootstrap/modal",
>>>>   "bootstrap/transition",
>>>> })
>>>> 
>>>> None of the 3 assets can be properly resolved... Not the end of the
>> world
>>>> of course :) Just a heads up.
>>>> 
>>>> Again I love this plugin ^^
>>>> 
>>>> -- Martin
>>>> 
>>>> On Tue, Feb 3, 2015 at 4:16 PM, Dmitry Gusev <dmitry.gu...@gmail.com>
>>>> wrote:
>>>> 
>>>>> Hi Martin,
>>>>> 
>>>>> My current goal is to avoid configuration files at this stage if
>>>> possible,
>>>>> but try to get all the data via static code analysis.
>>>>> 
>>>>> In this sense capabilities of the plugin are somewhat limited, for
>>>> example,
>>>>> this may be implemented with a little effort if you define your symbols
>>>> via
>>>>> SymbolSource configuration add/override (simple case without
>> conditional
>>>>> statements), something like:
>>>>> 
>>>>>   public static void contributeApplicationDefaults(
>>>>> 
>>>>>           MappedConfiguration<String, Object> configuration)
>>>>> 
>>>>>   {
>>>>> 
>>>>>       configuration.add("layout.scripts", "foo");
>>>>> 
>>>>>       configuration.add("layout.styles", Constants.BAR);
>>>>> 
>>>>>   }
>>>>> 
>>>>> Is this your case?
>>>>> 
>>>>> On Tue, Feb 3, 2015 at 7:45 AM, Martin Papy <martin.p...@gmail.com>
>>>> wrote:
>>>>> 
>>>>>> That is indeed a GREAT work :) Awesome :)
>>>>>> 
>>>>>> If I have 1 quick idea, it would be to be able to configure a symbol
>>>> list
>>>>>> so that we can open properly assets from the Import Annotation :).
>> Here
>>>>>> bellow I would like to defined what ${layout.scripts} points to.
>>>>>> 
>>>>>> @Import(library={
>>>>>>       "${layout.scripts}/html5shiv-printshiv.js"
>>>>>> },
>>>>>> stylesheet={
>>>>>>       "${layout.styles}/chronicles/bootstrap.css"
>>>>>> })
>>>>>> 
>>>>>> -- Martin
>>>>>> 
>>>>>> On Tue, Feb 3, 2015 at 10:01 AM, Bob Harner <bobhar...@gmail.com>
>>>> wrote:
>>>>>> 
>>>>>>> Awesome work, Dmitry! It just gets better and better...
>>>>>>> 
>>>>>>> On Mon, Feb 2, 2015 at 1:51 AM, Dmitry Gusev <dmitry.gu...@gmail.com
>>>>> 
>>>>>>> wrote:
>>>>>>>> Hey Tapestry Users,
>>>>>>>> 
>>>>>>>> I haven't updated release notes for the plugin for almost a year,
>>>>> lots
>>>>>> of
>>>>>>>> minor features & improvements were released since then.
>>>>>>>> 
>>>>>>>> Release 2.8.0 is different, because it introduces basic support for
>>>>>>>> Tapestry 5.4.
>>>>>>>> 
>>>>>>>> Some major changes:
>>>>>>>> 
>>>>>>>>  - Tapestry Context View
>>>>>>>>     - Now shows JavaScript modules, both imported via
>>>>>>>>     @Import(module="...") and required with
>>>>>>> JavaScriptSupport.require("...")
>>>>>>>>     - Validation rules for assets updated to include new
>>>> location:
>>>>>>>>     META-INF/assets
>>>>>>>> 
>>>>>>>> 
>>>>>>>>  - Tapestry Project Outline View
>>>>>>>>     - Added support for @ImportModule annotation
>>>>>>>>     - You can now see a list of services, decorators, advisors,
>>>> and
>>>>>>>>     contributors defined in each module. This is in addition to
>>>>>> Library
>>>>>>>>     Mappings & JavaScript Stacks. All are clickable and
>>>> selectable
>>>>>>>>     - Double click to jump to definition
>>>>>>>>     - Selecting items in this view updates the JavaDoc view &
>>>>> source
>>>>>>>>     range in Java editor if open (handy for learning the source
>>>>> code)
>>>>>>>> 
>>>>>>>> As usual you can find updated README with screenshots and download
>>>>>>>> instructions on Github:
>>>>>>>> 
>>>>>>>> https://github.com/anjlab/eclipse-tapestry5-plugin#readme
>>>>>>>> 
>>>>>>>> 
>>>>>>>> --
>>>>>>>> Dmitry Gusev
>>>>>>>> 
>>>>>>>> AnjLab Team
>>>>>>>> http://anjlab.com
>>>>>>> 
>>>>>>> ---------------------------------------------------------------------
>>>>>>> To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org
>>>>>>> For additional commands, e-mail: users-h...@tapestry.apache.org
>>>>>>> 
>>>>>>> 
>>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> Dmitry Gusev
>>>>> 
>>>>> AnjLab Team
>>>>> http://anjlab.com
>>>>> 
>>>> 
>>> 
>>> 
>>> 
>>> --
>>> Dmitry Gusev
>>> 
>>> AnjLab Team
>>> http://anjlab.com
>> 
>> 
> 
> 
> -- 
> Dmitry Gusev
> 
> AnjLab Team
> http://anjlab.com

Reply via email to