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'= ;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't = think it's a great idea. =C2=A0If someone ever wants to get/set somethi= ng that has the name of a task column (e.g. "id"), it's hard = to tell what to do.</div> <div style><br></div><div style>So, it looks like we'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==--