Committed your patches. Thanks - looks great.
I'm having trouble creating JIRA items, so I'll describe it first and
create the Jira item later.
Why don't you take a look at the sign-up page. I've redone the bottom
part of the web ui but I can't change the formatting for the part
labeled "Basic User Information" (email, password, etc.) . Could you
try and dig out where this is? Look at the file
"trunk\server\src\main\webapp\sign-up.html" for details. The code that
creates the fields is buried somewhere either in the ESME scala or
deeper in the lift scala code. Irregardless it is ugly and I'd love to
change it.
<div>
<fieldset><legend><lift:loc>ui_sign_up_basic_info</lift:loc></legend>
<div>
<signup:credentials/>
</div>
</fieldset>
D.
On Tue, Sep 29, 2009 at 9:49 AM, Xuefeng Wu <[email protected]> wrote:
> I submitted patch for ESME-84Could you help to test it with the patch for
> ESME-72?
>
> And is there any issue I could contribute?
>
> On Tue, Sep 29, 2009 at 3:44 PM, Vassil Dichev <[email protected]> wrote:
>
>> Could you just send a diff file, because it's hard to see what the
>> changes are when code's simply pasted in the mail body.
>>
>> Thanks!
>>
>>
>> On Mon, Sep 28, 2009 at 5:48 PM, Xuefeng Wu <[email protected]> wrote:
>> > Hi Richard,
>> > I'm sorry that I get some trouble when do ESME-84.
>> > *Could anybody help to finish this task?*
>> >
>> > I added some code in AccessPoolMgr. displayPoolUsers.
>> > I know it's ugly. :(
>> >
>> > def displayPoolUsers(in: NodeSeq): NodeSeq = {
>> > // get the span name to update
>> > val spanName = S.attr("the_id") openOr "PoolSpan"
>> > // get the current user
>> > val user = User.currentUser
>> >
>> > def doRender(): NodeSeq =
>> > Privilege.findAll(By(Privilege.pool, poolId.is)) match {
>> > case Nil => NodeSeq.Empty
>> > case xs => {
>> > bind("pool", in,
>> > "user" ->
>> > (lst => xs.flatMap(i => bind("user", lst,
>> > "name" ->
>> > User.find(i.user).map(
>> > _.
>> nickname.is
>> > ).getOrElse(""),
>> > "privilege" ->
>> > i.permission.is.toString
>> > ))))
>> > }
>> > }
>> > def updateSpan(): JsCmd = SetHtml(spanName, doRender())
>> >
>> > val poolDetailSpanName = S.attr("the_detail_id") openOr
>> "poolDetailSpan"
>> > def doPoolDetailRender():NodeSeq = AccessPool.find(poolId.is) match {
>> > case Full(ap) => bind(
>> > "pool", in,
>> > "name" -> ap.getName,
>> > "createdDate" -> ap.createdDate)
>> > case _ => NodeSeq.Empty
>> > }
>> > def updatePoolDetailSpan(): JsCmd = SetHtml(poolDetailSpanName,
>> > doPoolDetailRender())
>> >
>> > def updateAllSpan():JsCmd = SetHtml(spanName, doRender()) & JsRaw(";")
>> &
>> > SetHtml(poolDetailSpanName, doPoolDetailRender())
>> > updatePool.set(updateAllSpan)
>> > doRender
>> > }
>> >
>> >
>> > And modify pools_view/index.html
>> > <span id="poolDetailSpan">
>> > <lift:ignore>
>> > <!--
>> > The poolInfo snippet *MUST* appear on the page before
>> > the editPool snippet?
>> > -->
>> > </lift:ignore>
>> > <!-- Display Pool Basic Info -->
>> > <lift:poolUsers the_detail_id="poolDetailSpan">
>> > <table>
>> > <thead>
>> > <tr> <th>Name</th> <th>CreateDate</th> </tr>
>> > </thead>
>> >
>> > <tbody>
>> >
>> > <tr> <td><pool:name/></td> <td><pool:createdDate/></td> </tr>
>> > </tbody>
>> > </table>
>> > </lift:poolUsers>
>> > </span>
>> >
>> > <span id="PoolSpan">
>> >
>> > <lift:poolUsers the_id="PoolSpan">
>> > <table>
>> > <thead>
>> > <tr> <th>Name</th> <th>CreateDate</th> </tr>
>> > </thead>
>> > <tbody>
>> > <tr> <td><pool:name/></td> <td><pool:createdDate/></td> </tr>
>> > </tbody>
>> > </table>
>> >
>> > <table>
>> > <thead>
>> > <tr> <th>User</th> <th>Privilege</th> </tr>
>> > </thead>
>> >
>> > <tbody>
>> > <pool:user>
>> > <tr> <td><user:name/></td> <td><user:privilege/></td> </tr>
>> > </pool:user>
>> > </tbody>
>> > </table>
>> > </lift:poolUsers>
>> > </span>
>> >
>> >
>> > It's so ugly and It's work on Chrome and Safari, but do not work on
>> Firefox!
>> >
>> >
>> > The response is :
>> >
>> > try{jQuery("#PoolSpan").each(function(i) {this.innerHTML = "\u000a
>> > <table>\u000a <thead>\u000a <tr> <th>Name</th>
>> > <th>CreateDate</th> </tr>\u000a </thead>\u000a
>> > <tbody>\u000a <tr> <td><pool:name></pool:name></td>
>> > <td><pool:createdDate></pool:createdDate></td> </tr>\u000a
>> > </tbody>\u000a </table>\u000a \u000a <table>\u000a
>> > <thead>\u000a <tr> <th>User</th> <th>Privilege</th>
>> > </tr>\u000a </thead>\u000a \u000a <tbody>\u000a
>> > \u000a <tr> <td>admin</td> <td>Admin</td>
>> > </tr>\u000a \u000a </tbody>\u000a </table>\u000a
>> > ";});} catch (e) {};;try{jQuery("#poolDetailSpan").each(function(i)
>> > {this.innerHTML = "\u000a <table>\u000a <thead>\u000a
>> > <tr> <th>Name</th> <th>CreateDate</th> </tr>\u000a
>> > </thead>\u000a <tbody>\u000a <tr> <td>lift</td>
>> > <td>Mon, 28 Sep 2009 09:23:30 UTC</td> </tr>\u000a </tbody>\u000a
>> > </table>\u000a \u000a <table>\u000a <thead>\u000a
>> > <tr> <th>User</th> <th>Privilege</th> </tr>\u000a
>> > </thead>\u000a \u000a <tbody>\u000a
>> > <pool:user>\u000a <tr> <td><user:name></user:name></td>
>> > <td><user:privilege></user:privilege></td> </tr>\u000a
>> > </pool:user>\u000a </tbody>\u000a </table>\u000a ";});}
>> > catch (e) {}
>> >
>> >
>> >
>> > On Mon, Sep 28, 2009 at 8:40 PM, Richard Hirsch <[email protected]
>> >wrote:
>> >
>> >> I'll wait to add your ESME-72 code until you have finished the
>> >> ESME-84-related code. Otherwise, I wouldn't be able to test it.
>> >>
>> >> D.
>> >>
>> >> On Mon, Sep 28, 2009 at 2:21 PM, Xuefeng Wu <[email protected]> wrote:
>> >> > Thank you
>> >> >
>> >> > On Mon, Sep 28, 2009 at 7:42 PM, Vassil Dichev <[email protected]>
>> >> wrote:
>> >> >
>> >> >> First of all, welcome to the team, Xuefeng! I hope you have fun.
>> >> >>
>> >> >> Yes, AccessPoolMgr is the right place to check and edit (Dick, you're
>> >> >> getting better with Scala!)
>> >> >>
>> >> >> Good luck,
>> >> >> Vassil
>> >> >>
>> >> >>
>> >> >> On Mon, Sep 28, 2009 at 2:22 PM, Richard Hirsch <
>> [email protected]>
>> >> >> wrote:
>> >> >> > Yes take a look at "displayPoolUsers" in AccessPoolMgr.
>> >> >> >
>> >> >> > D.
>> >> >> >
>> >> >> > On Mon, Sep 28, 2009 at 1:06 PM, Xuefeng Wu <[email protected]>
>> wrote:
>> >> >> >> The poolUsers snippet code is in AccessPoolMgr?
>> >> >> >>
>> >> >> >> On Mon, Sep 28, 2009 at 6:56 PM, Richard Hirsch <
>> >> [email protected]
>> >> >> >wrote:
>> >> >> >>
>> >> >> >>> Part of the code is here:
>> >> >> >>> trunk\server\src\main\webapp\pools_view\index.html.
>> >> >> >>>
>> >> >> >>> Currently, the users for the poor are being added with this code:
>> >> >> >>>
>> >> >> >>> <lift:poolUsers the_id="PoolSpan">
>> >> >> >>> <table>
>> >> >> >>> <thead>
>> >> >> >>> <tr> <th>User</th> <th>Privilege</th> </tr>
>> >> >> >>> </thead>
>> >> >> >>>
>> >> >> >>> <tbody>
>> >> >> >>> <pool:user>
>> >> >> >>> <tr> <td><user:name/></td> <td><user:privilege/></td>
>> >> </tr>
>> >> >> >>> </pool:user>
>> >> >> >>> </tbody>
>> >> >> >>> </table>
>> >> >> >>>
>> >> >> >>> You have to have something similar (change HTML file and Scala
>> code
>> >> >> >>> for the pool details.
>> >> >> >>>
>> >> >> >>> D.
>> >> >> >>>
>> >> >> >>> On Mon, Sep 28, 2009 at 12:52 PM, Xuefeng Wu <[email protected]>
>> >> wrote:
>> >> >> >>> > I think so, but I'm not sure which view for AccessPool.
>> >> >> >>> > org.apache.esme.view.UserView?
>> >> >> >>> >
>> >> >> >>> > On Mon, Sep 28, 2009 at 6:43 PM, Richard Hirsch <
>> >> >> [email protected]
>> >> >> >>> >wrote:
>> >> >> >>> >
>> >> >> >>> >> There is already a view for the AccesPools. I'd try to use
>> that
>> >> as
>> >> >> >>> >> much as possible rather than putting any UI-related code in
>> the
>> >> >> >>> >> AccessPoolMgr.
>> >> >> >>> >>
>> >> >> >>> >> @vassil: what do you think?
>> >> >> >>> >>
>> >> >> >>> >> D.
>> >> >> >>> >>
>> >> >> >>> >> On Mon, Sep 28, 2009 at 11:42 AM, Xuefeng Wu <
>> [email protected]>
>> >> >> wrote:
>> >> >> >>> >> > Maybe I find it.Should I place code at
>> >> >> >>> org.apache.esme.lib.AccessPoolMgr?
>> >> >> >>> >> >
>> >> >> >>> >> > On Mon, Sep 28, 2009 at 5:38 PM, Xuefeng Wu <
>> [email protected]>
>> >> >> wrote:
>> >> >> >>> >> >
>> >> >> >>> >> >> Should we have a new snippet or view for AcessPool?
>> >> >> >>> >> >> I'm not sure where could place the bind code for display
>> >> >> AcessPoll
>> >> >> >>> >> detail.
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >> P.S. I submit a new patch for ESME-72, please replay the
>> new
>> >> one
>> >> >> if
>> >> >> >>> you
>> >> >> >>> >> >> download before, and I deleted the old patch file for
>> >> ignoring
>> >> >> >>> confuse.
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >> On Mon, Sep 28, 2009 at 3:14 PM, Richard Hirsch <
>> >> >> >>> [email protected]
>> >> >> >>> >> >wrote:
>> >> >> >>> >> >>
>> >> >> >>> >> >>> Looks good.
>> >> >> >>> >> >>>
>> >> >> >>> >> >>> I'll test it on the stax deployment.
>> >> >> >>> >> >>>
>> >> >> >>> >> >>> I've added a new JIRA item
>> >> >> >>> >> >>> (https://issues.apache.org/jira/browse/ESME-84) to
>> display
>> >> the
>> >> >> new
>> >> >> >>> >> >>> values in the UI. Why don't you take a crack at them. It
>> >> doesn't
>> >> >> ave
>> >> >> >>> >> >>> to be pretty (that will come later) just functional.
>> >> >> >>> >> >>>
>> >> >> >>> >> >>> D.
>> >> >> >>> >> >>>
>> >> >> >>> >> >>> On Mon, Sep 28, 2009 at 8:14 AM, Xuefeng Wu <
>> >> [email protected]>
>> >> >> >>> wrote:
>> >> >> >>> >> >>> > Thank you !
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>> > I'm in Shanghai now.
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>> > On Mon, Sep 28, 2009 at 1:58 PM, J Aaron Farr <
>> >> >> [email protected]>
>> >> >> >>> >> wrote:
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>> >> On Mon 28 Sep 2009 13:01, Xuefeng Wu <[email protected]
>> >
>> >> >> wrote:
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> > I write some code for ESME-72, but I don't know where
>> >> >> should I
>> >> >> >>> >> >>> submit?
>> >> >> >>> >> >>> >> >
>> >> >> >>> >> >>> >> > The attachment is the patch file, Thanks!
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> 0. If you don't have a JIRA account, create one here:
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >>
>> >> https://issues.apache.org/jira/secure/Signup!default.jspa
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> 1. Go to the ESME-72 issue page:
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> https://issues.apache.org/jira/browse/ESME-72
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> 2. Click the "Attach file to this issue" link on the
>> left
>> >> >> side.
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> 3. Attach your patch, include a comment, make sure you
>> >> click:
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> "Grant license to ASF for inclusion in ASF works (as
>> per
>> >> >> the
>> >> >> >>> >> Apache
>> >> >> >>> >> >>> >> License §5)"
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> If you have any trouble, please let us know.
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> Where are you in China? I'm currently in Guangzhou.
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >> --
>> >> >> >>> >> >>> >> J. Aaron Farr
>> >> >> >>> >> >>> >> 馮傑仁
>> >> >> >>> >> >>> >> www.cubiclemuses.com
>> >> >> >>> >> >>> >>
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>> > --
>> >> >> >>> >> >>> > Global R&D Center,Shanghai China,Carestream Health, Inc.
>> >> >> >>> >> >>> > Tel:(86-21)3852 6101
>> >> >> >>> >> >>> >
>> >> >> >>> >> >>>
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >>
>> >> >> >>> >> >> --
>> >> >> >>> >> >> Global R&D Center,Shanghai China,Carestream Health, Inc.
>> >> >> >>> >> >> Tel:(86-21)3852 6101
>> >> >> >>> >> >>
>> >> >> >>> >> >
>> >> >> >>> >> >
>> >> >> >>> >> >
>> >> >> >>> >> > --
>> >> >> >>> >> > Global R&D Center,Shanghai China,Carestream Health, Inc.
>> >> >> >>> >> > Tel:(86-21)3852 6101
>> >> >> >>> >> >
>> >> >> >>> >>
>> >> >> >>> >
>> >> >> >>> >
>> >> >> >>> >
>> >> >> >>> > --
>> >> >> >>> > Global R&D Center,Shanghai China,Carestream Health, Inc.
>> >> >> >>> > Tel:(86-21)3852 6101
>> >> >> >>> >
>> >> >> >>>
>> >> >> >>
>> >> >> >>
>> >> >> >>
>> >> >> >> --
>> >> >> >> Global R&D Center,Shanghai China,Carestream Health, Inc.
>> >> >> >> Tel:(86-21)3852 6101
>> >> >> >>
>> >> >> >
>> >> >>
>> >> >
>> >> >
>> >> >
>> >> > --
>> >> > Global R&D Center,Shanghai China,Carestream Health, Inc.
>> >> > Tel:(86-21)3852 6101
>> >> >
>> >>
>> >
>> >
>> >
>> > --
>> > Global R&D Center,Shanghai China,Carestream Health, Inc.
>> > Tel:(86-21)3852 6101
>> >
>>
>
>
>
> --
> Global R&D Center,Shanghai China,Carestream Health, Inc.
> Tel:(86-21)3852 6101
>