Model_Task function names

Shad Laws <shad-xpYdmXCiSuZWk0Htik3J/[email protected]> Fri, 3 May 2013 22:05:24 +0200
Newsgroups gmane.comp.web.gallery.devel
Message-ID <CA+z51A70AOwpiji7uFD+47bS34u2otW-=rBDngy4oiFdEzL2KQ@mail.gmail.com>
--===============6919734830666657877==
Content-Type: multipart/alternative; boundary=089e0158c3e06119be04dbd5e09b

--089e0158c3e06119be04dbd5e09b
Content-Type: text/plain; charset=UTF-8

Hey everyone,

While working on the photo/movie/album add/edit forms, I wanted to get
server_add working so I can add and test things, and found a problem:

We used get() and set() in the task model to get/set things in the task's
content column.  This totally clobbers the K3 ORM, which has a get() and
set() of its own.  Bummer.

I briefly looked into overloading the ORM get() and set(), but I don't
think it's a great idea.  If someone ever wants to get/set something that
has the name of a task column (e.g. "id"), it's hard to tell what to do.

So, it looks like we'll have to rename them.  Perhaps something like:
get_task(), set_task()
get_context(), set_context()
read(), write()
get_data(), set_data()
...?

Thoughts?

Take care,
Shad

--089e0158c3e06119be04dbd5e09b
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Hey everyone,<div><br></div><div style>While working on th=
e photo/movie/album add/edit forms, I wanted to get server_add working so I=
 can add and test things, and found a problem:</div><div style><br></div><d=
iv style>

We used get() and set() in the task model to get/set things in the task&#39=
;s content column. =C2=A0This totally clobbers the K3 ORM, which has a get(=
) and set() of its own. =C2=A0Bummer.</div><div style><br></div><div style>=
I briefly looked into overloading the ORM get() and set(), but I don&#39;t =
think it&#39;s a great idea. =C2=A0If someone ever wants to get/set somethi=
ng that has the name of a task column (e.g. &quot;id&quot;), it&#39;s hard =
to tell what to do.</div>

<div style><br></div><div style>So, it looks like we&#39;ll have to rename =
them. =C2=A0Perhaps something like:</div><div style>get_task(), set_task()<=
/div><div style>get_context(), set_context()</div><div style>read(), write(=
)</div>

<div style>get_data(), set_data()</div><div style>...?</div><div style><br>=
</div><div style>Thoughts?</div><div style><br></div><div style>Take care,<=
/div><div style>Shad</div></div>

--089e0158c3e06119be04dbd5e09b--


--===============6919734830666657877==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite
It's a free troubleshooting tool designed for production
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap2
--===============6919734830666657877==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

__[ g a l l e r y - d e v e l ]_________________________

[ list info/archive --> http://gallery.sf.net/lists.php ]
[ gallery info/FAQ/download --> http://gallery.sf.net ]
--===============6919734830666657877==--