[Seaside-dev] Re: [Seaside Commits] Seaside 3.0: Seaside-Tools-Web-dkh.54.mcz

[Julian Fitzell]

Our usual pattern for this would be "isNil ifFalse:"...

Julian

On Fri, Nov 13, 2009 at 11:15 AM,
<squeak-dev-noreply at lists.squeakfoundation.org> wrote:
> Dale Henrichs uploaded a new version of Seaside-Tools-Web to project Seaside 3.0:
> http://www.squeaksource.com/Seaside30/Seaside-Tools-Web-dkh.54.mcz
>
> ==================== Summary ====================
>
> Name: Seaside-Tools-Web-dkh.54
> Author: dkh
> Time: 13 November 2009, 11:15:41 am
> UUID: 79335fa6-65cd-480b-ade0-728823c053cd
> Ancestors: Seaside-Tools-Web-jf.53
>
> - eliminate use of #ifNotNil:
>
> =============== Diff against Seaside-Tools-Web-jf.53 ===============
>
> Item was changed:
>  ----- Method: WAFileAttributeEditor>>renderConfigurationOn: (in category 'rendering') -----
>  renderConfigurationOn: html
>        | selectedLib |
>        selectedLib := WAValueHolder new.
>        html form: [
>                html table: [
>                 html tableRow: [
>                        html tableData: [
>                                        html select
>                                                list: self libraries;
>                                                callback: [:value | selectedLib contents: value ];
>                                                size: 10 ].
>                        html tableData: [
>                                html anchor
>                                                title: 'Configure selected file library';
>                                                class: 'large';
>                                                onClick: 'return submitForm(this)';
> +                                               callback: [ selectedLib contents ~~ nil ifTrue: [self configureLibrary: selectedLib contents default ] ];
> -                                               callback: [ selectedLib contents ifNotNil: [self configureLibrary: selectedLib contents default ] ];
>                                                with: 'Configure'] ] ] ]!
>
>
> _______________________________________________
> commits mailing list
> To unsubscribe, email commits-leave at lists.seaside.st
> http://lists.seaside.st/listinfo/commits
>