Saturday, 2012-11-10

*** Quits: Vooch2 (~msalvucci@71-88-193-144.dhcp.kgpt.tn.charter.com) (*.net *.split)00:09
*** Joins: Vooch2 (~msalvucci@71-88-193-144.dhcp.kgpt.tn.charter.com)00:11
*** Quits: kirillka (~Miranda@99-83-52-95.baltnet.ru) (Quit: kirillka)00:20
*** Joins: dhx1 (~dhx1@60-241-62-124.tpgi.com.au)01:28
*** Joins: dregad (~dregad@203-236.193-178.cust.bluewin.ch)02:07
*** Joins: Paul_46 (~IceChat09@cpc1-enfi15-2-0-cust580.hari.cable.virginmedia.com)07:51
*** Joins: e66 (~quassel@27.147.181.37)09:52
*** Quits: giallu (~giallu@fedora/giallu) (Ping timeout: 246 seconds)14:28
*** Quits: dhx1 (~dhx1@60-241-62-124.tpgi.com.au) (Quit: Leaving)15:57
*** Quits: sdfjkljkdfsljkl (~sdfjkljkd@static.96.23.63.178.clients.your-server.de) (Remote host closed the connection)16:00
*** Joins: sdfjkljkdfsljkl (~sdfjkljkd@static.96.23.63.178.clients.your-server.de)16:00
GitHub48[mantisbt] dregad pushed 12 new commits to master-1.2.x: http://git.io/bxCRzQ16:13
GitHub48mantisbt/master-1.2.x a8f5420 Damien Regad: Re-indent complex if statement for better readability16:13
GitHub48mantisbt/master-1.2.x a7f9feb Damien Regad: Restrict $g_allow_reporter_* to users with reporter access...16:13
GitHub48mantisbt/master-1.2.x 9089f5e Damien Regad: Fix whitespace16:13
Paul_46dregad: moo16:27
dregadhey16:27
Paul_46do we really need to do whitespace fixes in 1.216:28
Paul_46i mean, it's not helping16:28
Paul_46;)16:28
dregadIt's a habit16:29
dregadand my editor is configured to trim trailing whitespace and to use tabs, which tends to "fix" whitespace semi automatically16:30
dregadTo make things easier, I try as much as possible, to commit whitespace changes separately from the rest16:30
dregadIn most cases, whitespace in 1.2 has already been fixed in 1.3/master16:31
Paul_46mm16:31
Paul_46also16:32
Paul_46+ - new 'EVENT_UPDATE_BUG_STATUS_FORM' plugin event16:32
Paul_46can we discuss that16:33
Paul_46 For example, there's a new 'json' api that isn't used (16:36
Paul_46> https://github.com/mantisbt/mantisbt/blob/master-1.2.x/core/json_api.php )16:36
Paul_46> within the core code.16:36
Paul_46Yes, I've introduced to prototype its usage in a plugin [1] to provide16:36
Paul_46in-page validation . If I'm happy with how this turns out from a16:36
Paul_46UI/code point of view I will consider porting it to master-1.2.x .16:36
Paul_46also16:36
Paul_46did you exclude that?16:36
dregadre: the new event - it's something that already existed in master16:39
dregadthere was a request from a user to add it16:39
dregadso i just backported16:39
dregadregarding the json - not sure what your point is16:39
Paul_46I dont think we should be adding api's in minor releases16:40
Paul_46that imo we might like to scrap16:40
dregadtalk to the author then16:40
dregadand revert the commit16:40
Paul_46[12:36.43] <Paul_46> in-page validation . If I'm happy with how this turns out from a16:40
Paul_46[12:36.43] <Paul_46> UI/code point of view I will consider porting it to master-1.2.x .16:40
Paul_46rombert seemed to suggest he'd not put it in master-1.216:40
dregadtbh, i did not double-check all commits or re-test the whole thing16:41
Paul_46tbh, your idea of forking mantis sounds more interesting16:41
dregadi would tend to agree with you on *really* freezing 1.216:42
dregadand focusing improvements on a new release16:42
dregadalong the lines of what vboctor wrote on the ml16:42
dregadif by forking you mean starting a separate project, that is not really my idea16:43
Paul_46it's more16:43
dregadi just thought you were going to do it with the mantisbt2 thing16:44
Paul_46people are forcing features into 1.216:44
Paul_46which makes it harder to move forwards16:44
dregadi agree16:44
Paul_46as in some cases16:44
Paul_46we've said before16:44
dregadbut that's a consequence of nothing happening in master for 2 years16:44
Paul_46we want to rewrite webservices from scratch16:44
Paul_46and db layer from scratch16:44
Paul_46so people "fixing" db layer+webservices16:44
Paul_46whilst cool16:44
dregadso the easy solution is to push new version out asap16:44
Paul_46also is making it hardr16:45
Paul_46I might just set up automated builds of 2.016:45
Paul_46and start runnign it at work16:45
Paul_46on commit, rebuild 2.0 and if it builds send mail+twitter notification16:45
Paul_46..17:00
Paul_46mm, irc client gone mad17:00
Paul_46dregad: check out mail I just sent to dev list17:14
dregadPaul_46 what should i say17:37
dregadyou're right obviously17:37
Paul_46well, even then, not sure I am17:37
Paul_46as I think an 'error code' in json rpc requires an ID17:37
Paul_46so we wouldn't be able to use *that* implementation as it is17:37
Paul_46basically my quesiton is17:38
Paul_46or more point is17:38
Paul_46a) i've not done much jquery stuff17:38
Paul_46b) i've been looking at json+jquery stuff recently for something else17:38
Paul_46c) i've not seen the construction of json that we've added when looking at b17:38
Paul_46which makes me wonder if there is some logic, or if we invented something17:39
Paul_46there's two ways I can look at this17:39
Paul_461) being i don't really care as we can scrap api in '2.x' or review it at least - and count 2.x as a 'fresh start'17:40
Paul_462) we should try and make migration path from 1.2 to a '2.x' somewhat feasible17:40
*** Quits: dregad (~dregad@203-236.193-178.cust.bluewin.ch) (Quit: For Sale: Parachute. Only used once, never opened, small stain.)17:54
GitHub74[mantisbt] dregad pushed 1 new commit to master-1.2.x: http://git.io/M3hwAw18:04
GitHub74mantisbt/master-1.2.x 23b9476 Damien Regad: Changed version to 1.2.13dev18:04
*** Quits: Paul_46 (~IceChat09@cpc1-enfi15-2-0-cust580.hari.cable.virginmedia.com) (Quit: Do fish get thirsty?)18:13

Generated by irclog2html.py 2.10.0 by Marius Gedminas - find it at mg.pov.lt!