My favorites | Sign in
Logo
          
New issue | Search
for
| Advanced search | Search tips
Issue 86: Generating <form> without extra fields
  Back to list
Status:  Fixed
Owner:  mykdavies
Closed:  Aug 2008
Type-Feature
Priority-Medium
Version-Seaside2.9


Sign in to add a comment
 
Reported by renggli, Jul 04, 2008
I had run across a case where I needed to generate a form that didn't
have built-in helpers for _s and _k, so I ended up with the following,

Smalltalk.Seaside defineClass: #WABasicFormTag
       superclass: #{Seaside.WAFormTag}
       indexedType: #none
       private: false
       instanceVariableNames: ''
       classInstanceVariableNames: ''
       imports: ''
       category: ''

Seaside.WABasicFormTag>>after
 "This method is intentionally empty"

Seaside.WARenderCanvas>>basicForm
 ^self brush: WABasicFormTag new.

Obviously, this could be better suited to become a feature of standard
WAFormTag, but this is a simpler extension maintenance-wise for me. Was
hoping some form of this could be considered for inclusion in base.

Thanks!

-Boris

Comment 1 by bpopov, Jul 04, 2008
Ideally the "basic" form would be a superclass and a specialized "seaside" form
subclass with additional defaultAction and session tracking behavior.
Comment 3 by mykdavies, Aug 30, 2008
fixed as proposed

--------------------------------
Name: Seaside-Core-MAD.195
Author: MAD
Time: 30 August 2008, 3:09:08 pm
UUID: d44d13a4-7694-11dd-89de-000a95da90cc
Ancestors: Seaside-Core-pmm.194

- created WABasicForm as new superclass of WAForm.
- also fixed typo in name of WAFormTag>>defaultUrlCharset

 Issue 86 


--------------------------------
Name: Seaside-Tests-MAD.148
Author: MAD
Time: 30 August 2008, 3:11:09 pm
UUID: 1cc1b3e2-7695-11dd-89de-000a95da90cc
Ancestors: Seaside-Tests-pmm.147

- added testBasicForm

 Issue 86 


-------------------------------
Name: Seaside-Core-MAD.196
Author: MAD
Time: 30 August 2008, 3:24:23 pm
UUID: f6383078-7696-11dd-89de-000a95da90cc
Ancestors: Seaside-Core-MAD.195

Corrected class comments for WAFormTag, WABasicFormTag

Comment 4 by philippe.marschall, Aug 30, 2008
Name: Seaside-Core-MAD.195
Author: MAD
Time: 30 August 2008, 3:09:08 pm
UUID: d44d13a4-7694-11dd-89de-000a95da90cc
Ancestors: Seaside-Core-pmm.194

- created WABasicForm as new superclass of WAForm.
- also fixed typo in name of WAFormTag>>defaultUrlCharset

 Issue 86 
Status: Fixed
Owner: mykdavies
Labels: Version-Seaside2.9
Sign in to add a comment

Hosted by Google Code