[
10:17am
]
balunasj_
:
ok guys lets get back at it
[
10:17am
]
balunasj_
:
we have our regular team meeting in about 45 min
[
10:18am
]
balunasj_
:
the second half of today we wanted to discuss CDK update to status, estimates and remaining tasks for A2 - http://community.jboss.org/wiki/RichFaces40ALPHA2CDKStatus
[
10:19am
]
balunasj_
:
alexsmirnov: nbelaevski : please get us started by discussing what has been updated from the last meeting, and what you think is required for A2?
[
10:20am
]
nbelaevski
:
I did the following updates:
[
10:20am
]
nbelaevski
:
- converted RF-7732 (CDK template compiler) from subtask into a task and added subtasks there
[
10:20am
]
nbelaevski
:
- updated wiki with the estimates
[
10:21am
]
nbelaevski
:
- marked the things that in my opinion can be slipped to next release on the wiki
[
10:21am
]
pmuir left the chat room. (Read error: 110 (Connection timed out))
[
10:21am
]
Konstantin1 left the chat room. (Remote closed the connection)
[
10:22am
]
Konstantin1 joined the chat room.
[
10:22am
]
nbelaevski
:
tasks that are marked as the ones that can be slipped are also for discussion
[
10:22am
]
balunasj_
:
excellent!
[
10:23am
]
balunasj_
:
before digging in - alexsmirnov what updates where you able to do and where?
[
10:23am
]
nbelaevski
:
so I would like balunasj_: alexsmirnov: to take a look at this and discuss
[
10:24am
]
alexsmirnov
:
Currently, CDK generates component and renderer classes and faces-config.xml. Other parts ( Behavior, Validator, converter etc ) is near to ready.
[
10:25am
]
alexsmirnov
:
The most important thing that is necessary to have working library is taglib/taghandler generator that is developed now.
[
10:25am
]
balunasj_
:
alexsmirnov: where you able to come up with estimates and such so I can plug into project plan?
[
10:26am
]
alexsmirnov
:
I moved CDK from custom context/factories to dependency injection ( JSR-330/Google Guice ) that simplifies project structure.
[
10:27am
]
alexsmirnov
:
I think taglib end remaining classes require 1-2 weeks for development. Minor tasks may take 1-2 weeks more.
[
10:28am
]
balunasj_
:
alexsmirnov: and these are in parallel with the template compiler work nbelaevski outlined?
[
10:28am
]
alexsmirnov
:
Yes, we work in parrallel.
[
10:29am
]
balunasj_
:
ok
[
10:29am
]
balunasj_
:
alexsmirnov: besides the tag lib, what other tasks do you foresee?
[
10:30am
]
balunasj_
:
alexsmirnov: I just want to understand tasks a little more.
[
10:32am
]
amarkhel left the chat room. (Read error: 110 (Connection timed out))
[
10:32am
]
pmuir_ is now known as pmuir.
[
10:33am
]
balunasj_
:
alexsmirnov: also is there a reason you choose 330/guice instead of cdi ( which can run in se)?
[
10:35am
]
alexsmirnov
:
We can use any JSR-330 implementation, but weld seems overcomplexity for CDK. Another reason that Maven 3.0 is going to use Guice instead of Plextor framework too.
[
10:36am
]
alexsmirnov
:
I use only javax.inject annotations only so it would use Weld too.
[
10:37am
]
balunasj_
:
alexsmirnov: I should have saved that question for later, I really want to focus on tasks and plans for A2
[
10:37am
]
balunasj_
:
alexsmirnov: ok - I am curious and will try to review.
[
10:39am
]
balunasj_
:
so member:alexsmirnov: besides the tag lib, what other tasks do you foresee for A2?
[
10:39am
]
alexsmirnov
:
http://java.dzone.com/articles/plexus-guice-why-guice
[
10:40am
]
alexsmirnov
:
Remaining classes generation and tests, including integration tests.
[
10:41am
]
balunasj_
:
alexsmirnov: are these all part of the 1-2 weeks more from above or other tasks?
[
10:43am
]
balunasj_
:
while alex is answering that lets move on to nbelaevski questions and points.
[
10:43am
]
balunasj_
:
from here - http://community.jboss.org/wiki/RichFaces40ALPHA2CDKStatus
[
10:43am
]
nbelaevski
:
balunasj_: ok
[
10:43am
]
alexsmirnov
:
Yes, the total estimate is about 4 weeks ( I've never seen before that optimistic estimates were correct ).
[
10:44am
]
balunasj_
:
nbelaevski: I agree Reporting of compilation errors via CDK could slip.
[
10:44am
]
balunasj_
:
alexsmirnov:   sure - I'm not asking for optimistic estimates, but just estimates that we can use for planning
[
10:44am
]
nbelaevski
:
balunasj_: ok, I'll remove it from the scope
[
10:44am
]
balunasj_
:
nbelaevski: concern is what might be "hidden" if this is not reported?
[
10:45am
]
balunasj_
:
will the info be available somewhere?
[
10:45am
]
nbelaevski
:
balunasj_: no, build will fail
[
10:45am
]
balunasj_
:
will the failed build show the root cause?
[
10:45am
]
nbelaevski
:
balunasj_: yes, but it's a hard way of reporting
[
10:45am
]
balunasj_
:
  yes it is
[
10:45am
]
balunasj_
:
but for A2 that is fine
[
10:45am
]
balunasj_
:
I just wanted to make sure it was not hinding
[
10:46am
]
balunasj_
:
hiding
[
10:46am
]
balunasj_
:
any
[
10:46am
]
nbelaevski
:
balunasj_: sure, it shouldn't be doing so
[
10:46am
]
balunasj_
:
ok
[
10:47am
]
balunasj_
:
The same attributes declared in template - this will be important for a Beta release
[
10:47am
]
balunasj_
:
especially when we start talking about
[
10:47am
]
balunasj_
:
tools integration
[
10:47am
]
balunasj_
:
but agree that can be out of scope for A2
[
10:48am
]
nbelaevski
:
we talked with alexsmirnov about this,,, template attributes seem to be important because it's the way to separate renderer-dependent and renderer-independent attributes
[
10:48am
]
balunasj_
:
nbelaevski: for the disabled components does disabled == rendered="false" ?
[
10:48am
]
nbelaevski
:
balunasj_: no, events handler are not rendered and component is not decoded
[
10:49am
]
nbelaevski
:
balunasj_: e.g. disabled commandLink is represented as SPAN
[
10:49am
]
balunasj_
:
nbelaevski: ok
[
10:49am
]
balunasj_
:
wanted to make sure I understood you point.
[
10:50am
]
balunasj_
:
back to the attributes
[
10:50am
]
balunasj_
:
nbelaevski: does this mean they are needed for A2 afterall?
[
10:50am
]
nbelaevski
:
alexsmirnov: ^ - can you please clarify?
[
10:51am
]
alexsmirnov
:
balunasj: We need a some place for rendered-specific attributes and events. Keep them all together with renderer template seems most useful.
[
10:52am
]
alexsmirnov
:
Although they could be also defined in renderer-related component or faces-config.xml.
[
10:53am
]
balunasj_
:
I seeing your point though - it might be easier for development if attributes were local to template
[
10:53am
]
balunasj_
:
would the faces-config then be populated from this, or would both be needed?
[
10:54am
]
balunasj_
:
nbelaevski: just so we have time - the forEach deferred makes sense if we don't need it yet
[
10:55am
]
nbelaevski
:
balunasj_: and we don't
[
10:55am
]
alexsmirnov
:
We generate faces-config with all renderer attributes wherever they has been defined.
[
10:55am
]
balunasj_
:
nbelaevski: and then for the templating reuse - my question would be how much refactoring might be needed later if it is not available soon?
[
10:55am
]
balunasj_
:
alexsmirnov: ok
[
10:55am
]
balunasj_
:
good
[
10:55am
]
nbelaevski
:
also attributes can be included into *.taglib.xml
[
10:56am
]
balunasj_
:
we might want to be more strict on attribute definition - sometimes have too many places makes it hard to understand the diff
[
10:56am
]
balunasj_
:
even if there is nonw
[
10:56am
]
balunasj_
:
none
[
10:57am
]
pyaschenko joined the chat room.
[
10:57am
]
pyaschenko
:
Hi All
[
10:57am
]
prabhat joined the chat room.
[
10:57am
]
nbelaevski
:
balunasj_: we've developed 3.x without template code reuse at all, but that's a great enhancement I want to make templating code more maintenable...
[
10:58am
]
balunasj_
:
nbelaevski: I like reuse whenever possible and obviously DRY.
[
10:58am
]
nbelaevski
:
balunasj_: if we implement this feature at early B1 stage, then it's ok and no refactoring will be necessary
[
10:58am
]
nbelaevski
:
balunasj_: me too
[
10:58am
]
balunasj_
:
yeah that is just what I was thinking too
[
10:58am
]
balunasj_
:
we want it in place for people to use and understand early
[
10:58am
]
alexsmirnov
:
nbelaevski: taglib.xml has no place for attributes.
[
11:00am
]
nbelaevski
:
alexsmirnov: hm, IDE suggests to add them into tag attribute
[
11:01am
]
nbelaevski
:
alexsmirnov: also "attribute" element is present in web-facelettaglibrary_2_0.xsd
[
11:01am
]
alexsmirnov
:
nbelaevski: Thanks, I've missed that.
[
11:02am
]
balunasj_
:
nbelaevski: alexsmirnov : I think we are going to need to shelve this for now and take offline
[
11:02am
]
nbelaevski
:
alexsmirnov: I guess this make more sense for IDEs than faces-config.xml
[
11:02am
]
balunasj_
:
we should start the team meeting soon