<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On 16 Oct 2013, at 11:53, Pat @ Pgogy <<a href="mailto:xerte@pgogywebstuff.com">xerte@pgogywebstuff.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">How long would it take to change the code over to abstraction - honestly - I'd say 2-3 days (I internationalised over a weekend)</div></blockquote><div><br></div>Yes, and then :</div><div><br></div><div>a. There was refactoring and bug fixing by others for days/weeks afterwards</div><div><br></div><div>b. If there had been a discussion beforehand, perhaps a different (and perhaps better) solution might have been chosen.</div><div><br></div><div>I'm all for someone 'fixing' the mysql_ stuff - but there needs to be a short discussion beforehand. There are 101 different possible approaches - some are more equal than others etc.</div><div><br></div><div><br></div><div>There's a big difference between replacing instances of 'mysql_query' and associated stuff with db_query* versus making some sort of library/api.</div><div><br></div><div>The effort/changes required for a library/api would probably replace/rewrite/overwrite whatever may have been done changing things to use e.g. db_query*.</div><div><br></div><div><br></div><div>David.</div><div><br></div><div><br><blockquote type="cite"><div dir="ltr"><div><br></div><div>But it just feels a bit odd to just do the database stuff</div>
<div><br></div><div>I will do it for 9,999 pounds</div></div></blockquote></div><br><div apple-content-edited="true">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Pale Purple Ltd. (Company No: 5580814)</div><div>'Web and Mobile Application Development for Business'</div><div><br></div><div><a href="http://www.palepurple.co.uk">http://www.palepurple.co.uk</a> </div><div>Office: 0845 0046746 Mobile: 07792380669 </div><div><br></div><div>Follow us on Twitter: @PalePurpleLtd</div></div></span></div></span></div></span></div></span></div>
</div>
<br></body></html>