| ../irclogs/#mantishelp.2009-06-22.log | ||
| --- scribe started --- | 00:00 | |
| mantisbot | New bug: Bug 10610 - eliasn - open - new | 03:02 |
|---|---|---|
| mantisbot | New bug: Include information about test execution - http://www.mantisbt.org/bugs/view.php?id=10610 | 03:02 |
| mantisbot | New bug: Bug 10611 - Kirill - open - new | 07:33 |
| mantisbot | New bug: Attach not recognize correctly - http://www.mantisbt.org/bugs/view.php?id=10611 | 07:33 |
| dhx_m | looking towards the 1.2 release, there seems to be quite a few TODO things specifically marked for 1.2 in database_api | 08:08 |
| CIA-17 | Mantisbt: hickseydr * rb77fcd42ffb6 /docbook/developers/en/contributers.sgml: Update contributor guide for pending 1.2 release | 08:10 |
| [KK]Kirill | dhx_m: lo | 08:12 |
| dhx_m | [KK]Kirill: hi | 08:13 |
| [KK]Kirill | dhx_m: Something strange | 08:13 |
| [KK]Kirill | Can you take me your time? | 08:13 |
| dhx_m | whats up? :) | 08:14 |
| [KK]Kirill | about bug 10611 | 08:14 |
| mantisbot | Bug 10611 - Kirill - open - new | 08:14 |
| mantisbot | Attach not recognize correctly - http://www.mantisbt.org/bugs/view.php?id=10611 | 08:14 |
| dhx_m | hmm that's odd | 08:14 |
| [KK]Kirill | look | 08:15 |
| dhx_m | the live site at mantisbt.org uses the latest version | 08:15 |
| [KK]Kirill | core/file_api.php | 08:15 |
| dhx_m | so I'd expect it to be a problem there too | 08:15 |
| [KK]Kirill | function file_get_extension( $p_filename ) { | 08:15 |
| [KK]Kirill | I add string echo $p_filename; | 08:16 |
| [KK]Kirill | result SQL стратам.png | 08:16 |
| [KK]Kirill | after $i = utf8_strlen( $p_filename ) - 1; | 08:16 |
| [KK]Kirill | I place string | 08:17 |
| [KK]Kirill | echo $p_filename[$i]; | 08:17 |
| [KK]Kirill | result ? | 08:17 |
| [KK]Kirill | something strange with one char of string | 08:17 |
| [KK]Kirill | and other chars too | 08:18 |
| dhx_m | you're right, that function is broken with respect to UTF8 | 08:18 |
| dhx_m | I'll see if I can fix it | 08:19 |
| [KK]Kirill | I think more function broken. I have problem such this in other function, but I forgot where.. | 08:19 |
| [KK]Kirill | If you can solve this, I try found and fix other promblem | 08:20 |
| dhx_m | ok | 08:20 |
| CIA-17 | Mantisbt: hickseydr * r3758e6f12788 /library/README.libs: Update libraries README | 08:35 |
| CIA-17 | Mantisbt: hickseydr * r96814088450f /lang/langreadme.txt: Remove deprecated information from langreadme | 08:50 |
| [KK]Kirill | Oh.. I remember where I can see that | 08:56 |
| [KK]Kirill | I see in geshi | 08:56 |
| dhx_m | we use geshi? :D | 08:56 |
| [KK]Kirill | I think it's in my server broke :) | 08:57 |
| [KK]Kirill | not | 08:57 |
| [KK]Kirill | I use geshi in pastebin | 08:57 |
| paul_ | +ezc | ez Components | 2008.2.3 | unpatched | 09:00 |
| paul_ | dhx_m: please add correct version | 09:00 |
| paul_ | dhx_m: a) replace file_get_extension with a regex | 09:03 |
| [KK]Kirill | paul_: hi | 09:04 |
| paul_ | b) it's probably safe to use strlen itself given how we're trying to use it there | 09:04 |
| paul_ | hi | 09:06 |
| [KK]Kirill | paul_: why it's can be? | 09:07 |
| [KK]Kirill | string show normal, but each char broken in utf-8? | 09:07 |
| paul_ | I think i'd expect that to break | 09:10 |
| paul_ | instead of getting strlen, we're getting utf_strlen | 09:10 |
| paul_ | then recursing through each char of the string | 09:10 |
| paul_ | (which if php isn't treating as utf or something | 09:10 |
| paul_ | either way | 09:11 |
| paul_ | strikes me as something that could be written as a regex | 09:11 |
| paul_ | we only *need* the last N chars before a '.' | 09:11 |
| paul_ | normally 2 / 3 / 4 chars e.g. .html /.htm | 09:11 |
| [KK]Kirill | paul_: I ask in russian forum | 09:22 |
| [KK]Kirill | they answer that utf have 2 byte in string | 09:22 |
| [KK]Kirill | $p_filename[$i] - 1 byte | 09:23 |
| [KK]Kirill | I test in bug 10611 | 09:26 |
| mantisbot | Bug 10611 - Kirill - open - new | 09:26 |
| mantisbot | Attach not recognize correctly - http://www.mantisbt.org/bugs/view.php?id=10611 | 09:26 |
| [KK]Kirill | this file equal, but diff names | 09:26 |
| [KK]Kirill | Oh I fix it | 09:30 |
| [KK]Kirill | I change $p_filename[$i] to utf8_substr($p_filename,$i,1) | 09:30 |
| dhx_m | back | 10:03 |
| dhx_m | paul_: what is the correct version? | 10:03 |
| dhx_m | paul_: I just guessed you used the latest stable... I take it you used latest unstable? | 10:05 |
| dhx_m | do we have utf8 regex? | 10:05 |
| paul_ | erm | 10:05 |
| paul_ | kinda | 10:05 |
| paul_ | I used latest svn :P | 10:06 |
| dhx_m | ahhh :) | 10:06 |
| paul_ | which I think actually corresponds to 2009.1 beta1 | 10:06 |
| dhx_m | sounds good to me :) | 10:06 |
| paul_ | I got warned there was a commit in svn relating to large number of data points that I'd want) | 10:06 |
| paul_ | but in terms of file extension | 10:07 |
| paul_ | I reckon we'd be better doing a regex with /u or something | 10:08 |
| paul_ | it strikes me as a complicated function to get what (normally) will be last 3 chars of a string | 10:08 |
| dhx_m | regex seems like high overhead | 10:09 |
| dhx_m | anyway, I disagree with using the file extension | 10:10 |
| dhx_m | when we should be using the mime type | 10:10 |
| paul_ | tbh, regex might be less | 10:18 |
| paul_ | but yea | 10:18 |
| paul_ | that's another point | 10:18 |
| * paul_ thinks | 10:18 | |
| dhx_m | I think Regex is probably the best solution | 10:19 |
| dhx_m | because it is easiest | 10:19 |
| dhx_m | although technically it'd be better to do a backwards search through the string until you hit the first period (.) | 10:19 |
| dhx_m | assuming the file has an extension! :) | 10:19 |
| mantisbot | New bug: Bug 10612 - rombert - open - new | 10:24 |
| mantisbot | New bug: mc_project_get_issue_headers returns an empty response (this bugtracker) - http://www.mantisbt.org/bugs/view.php?id=10612 | 10:24 |
| mantisbot | New bug: Bug 10613 - nuan burke - open - new | 12:04 |
| mantisbot | New bug: Cannot Filter by Project - http://www.mantisbt.org/bugs/view.php?id=10613 | 12:04 |
| nuclear_eclipse | morning all | 12:23 |
| [KK]Kirill | lo | 12:23 |
| [KK]Kirill | John, how I can see what diff curent file with commit state? | 12:23 |
| [KK]Kirill | in git | 12:24 |
| nuclear_eclipse | `git diff filename` ? :P | 12:24 |
| [KK]Kirill | :) it's work. thanks | 12:27 |
| mantisbot | New bug: Bug 10614 - Kirill - open - new | 12:30 |
| mantisbot | New bug: assigned issue not assigned - http://www.mantisbt.org/bugs/view.php?id=10614 | 12:30 |
| dhx_m | hi nuclear_eclipse | 12:33 |
| nuclear_eclipse | hi dh | 12:33 |
| nuclear_eclipse | x | 12:33 |
| paul_ | nuclear_eclipse: I HATE PAGE NUMBERS | 13:27 |
| [KK]Kirill | paul_: You implement new bug class? | 13:29 |
| [KK]Kirill | or not? | 13:29 |
| nuclear_eclipse | hmm, what? | 13:32 |
| paul_ | you dont want to know | 13:32 |
| [KK]Kirill | bug 10614. I see, somebody did hard work to change bug class use | 13:32 |
| mantisbot | Bug 10614 - Kirill - open - new | 13:32 |
| mantisbot | assigned issue not assigned - http://www.mantisbt.org/bugs/view.php?id=10614 | 13:32 |
| [KK]Kirill | may be my issue is result of this work? | 13:34 |
| nuclear_eclipse | [KK]Kirill: I can't replicate the issue | 13:35 |
| [KK]Kirill | Can you create test project and give me manager right on it? | 13:37 |
| [KK]Kirill | in your issue? | 13:37 |
| [KK]Kirill | in your mantis | 13:37 |
| nuclear_eclipse | gimme a bit to update my live site to latest git master | 13:40 |
| [KK]Kirill | nuclear_eclipse: may be I create test project? | 13:40 |
| [KK]Kirill | I always use last git | 13:41 |
| dhx_m | it's paul_ 's fault for sure | 13:42 |
| dhx_m | :p | 13:42 |
| nuclear_eclipse | [KK]Kirill: perhaps it's a configuration issue? | 13:43 |
| dhx_m | nuclear_eclipse: what do we do with UTF8 regex matching? | 13:54 |
| nuclear_eclipse | depends | 13:54 |
| dhx_m | add /u ? | 13:54 |
| nuclear_eclipse | that's only part of it | 13:54 |
| nuclear_eclipse | a) /u will allow regexs to work on multibyte strings, b) to actually match utf8 characters, we need to be matching against more than a-zA-Z0-9... | 13:55 |
| dhx_m | true | 13:55 |
| dhx_m | although I'd say your regex is wrong if you're using ranges like a-Z | 13:56 |
| nuclear_eclipse | well, it all depends on the context... | 13:57 |
| dhx_m | I just don't see why you'd want to match against \d instead of 0-9 | 13:58 |
| dhx_m | I mean... match against 0-9 instead of \d | 13:58 |
| nuclear_eclipse | I'm lost | 13:59 |
| nuclear_eclipse | oh, nvm | 13:59 |
| nuclear_eclipse | your wording bugged out my brain :P | 13:59 |
| dhx_m | well I would assume that matching against \w would match any character that is a "word character" in the native UTF8 locale | 14:00 |
| dhx_m | whereas a-z specifically only matches a-z in the English locale | 14:00 |
| nuclear_eclipse | it's complicated, let's just put it that way :P | 14:00 |
| [KK]Kirill | dhx_m: I write how I fix it | 14:01 |
| [KK]Kirill | I use utf8_substr() | 14:01 |
| dhx_m | so if we're using regex on utf8 strings... it'll be bad? | 14:01 |
| nuclear_eclipse | yes/no - depends on what we're trying to do with it | 14:02 |
| dhx_m | nuclear_eclipse: take for example plugin_file.php:36 | 14:03 |
| dhx_m | that would prevent someone from using a Unicode filename | 14:03 |
| nuclear_eclipse | right | 14:03 |
| nuclear_eclipse | but at the same time, since many filesystems don't handle utf filenamse correctly anyways, I'm not considering that a giant loss atm... | 14:04 |
| dhx_m | for no real reason at all? (unless PHP file functions aren't UTF8 compatible... which I doubt) | 14:04 |
| dhx_m | on Windows I imagine it all completely breaks | 14:04 |
| nuclear_eclipse | right | 14:04 |
| nuclear_eclipse | that's the main reason I made that decision | 14:04 |
| dhx_m | but then we'd need to prevent all attachment names from containing Unicode characters? | 14:05 |
| nuclear_eclipse | Windows tends to break everything, so if we simply say that all plugins must be using ascii names, we ensure they'll work on any install | 14:05 |
| nuclear_eclipse | and we don't enforce utf8 on attachments because we store them in the database by default | 14:05 |
| [KK]Kirill | nuclear_eclipse: I talk about attach event | 14:06 |
| nuclear_eclipse | if someone wants to put attachments on disk, they will probably understand the problems involved | 14:06 |
| nuclear_eclipse | dhx_m: http://www.php.net/manual/en/regexp.reference.php | 14:06 |
| nuclear_eclipse | search for "utf" | 14:06 |
| dhx_m | yep | 14:07 |
| nuclear_eclipse | it would seem to me that \w is not utf-8 compatible | 14:07 |
| nuclear_eclipse | we'd have to start doing \pL and so forth | 14:07 |
| dhx_m | A "word" character is any letter or digit or the underscore character, that is, any character which can be part of a Perl "word". The definition of letters and digits is controlled by PCRE's character tables, and may vary if locale-specific matching is taking place. For example, in the "fr" (French) locale, some character codes greater than 128 are used for accented letters, and these are matched by \w. | 14:08 |
| nuclear_eclipse | right, that's my point | 14:08 |
| dhx_m | also they provide \C specifically to match single bytes, so I imagine \w would be UTF-8 compatible? | 14:08 |
| nuclear_eclipse | you can't rely on the locale | 14:08 |
| nuclear_eclipse | eg, if I set my server locale to iso8859, pcre won't match utf8 characters with \w | 14:09 |
| nuclear_eclipse | PHP's a sticy mess when it comes to locales and such | 14:09 |
| dhx_m | "Matching characters by Unicode property is not fast, because PCRE has to search a structure that contains data for over fifteen thousand characters. That is why the traditional escape sequences such as \d and \w do not use Unicode properties in PCRE. " | 14:10 |
| dhx_m | ah | 14:10 |
| nuclear_eclipse | like I said, it's complicated :P | 14:10 |
| dhx_m | that documentation is confusing... I guess the part about French accents was in reference to old school code pages for characters > 0d127 | 14:10 |
| nuclear_eclipse | right | 14:11 |
| * Reapazor looks nervously around -- i see were getting ready for a release...but api still is broken ? :/ | 14:11 | |
| nuclear_eclipse | what api is still broken? soap you mean? | 14:11 |
| Reapazor | ya | 14:12 |
| Reapazor | everything still coming back with access denied | 14:12 |
| Reapazor | (works in stable release 1.1) | 14:13 |
| Reapazor | keeps throwing an access denied exception to a user with full rights :) | 14:13 |
| dhx_m | nuclear_eclipse: what about $g_user_login_valid_regex = '/^[\w \-]+$/'; | 14:13 |
| nuclear_eclipse | Reapazor: I think either a) something is broken in your configuration, or b) your client is not up to date with any changes that may be in 1.2 | 14:13 |
| dhx_m | nuclear_eclipse: is that a candidate for UTF-8 regex? | 14:14 |
| nuclear_eclipse | dunno | 14:14 |
| dhx_m | I imagine you need mbstring though :( | 14:14 |
| nuclear_eclipse | Reapazor: bug 10612 acts as though everything works fine except for a couple functions themselves, so I can't imagine that it's totally broken... | 14:15 |
| mantisbot | Bug 10612 - rombert - open - assigned | 14:15 |
| mantisbot | mc_project_get_issue_headers returns an empty response (this bugtracker) - http://www.mantisbt.org/bugs/view.php?id=10612 | 14:15 |
| nuclear_eclipse | dhx_m: iirc, pcre's utf8 does not use mbstring, like mentioned, it has its own lookup tables for multibyte characters | 14:15 |
| dhx_m | nuclear_eclipse: ugh, what a mess :p | 14:16 |
| nuclear_eclipse | utf8 is always a mess in any language that wasn't built from scratch to make it not be a mess | 14:16 |
| dhx_m | Reapazor: does 1.2.0a3 work at all? | 14:17 |
| nuclear_eclipse | even in python 3, which supports utf8 from the ground up, utf8 is still a bit tricky at times | 14:17 |
| dhx_m | yep | 14:17 |
| dhx_m | once the world moves to UTF-8 it'll be a much happier place though :) | 14:17 |
| dhx_m | for instance, being able to match things like \Sc for a currency symbol is very cool :) | 14:18 |
| Reapazor | i just pulled from git repo this morning | 14:19 |
| nuclear_eclipse | no, once the world move completely to ASCII, it'll be amuch happier place :P | 14:19 |
| Reapazor | still no dice | 14:19 |
| Reapazor | have things changed in the auth process / | 14:19 |
| Reapazor | for soap ? | 14:19 |
| nuclear_eclipse | Reapazor: not afaik | 14:19 |
| dhx_m | Reapazor: 1.2.0a3 is from months ago, so I was wondering if any of the recent fixes caused the problem | 14:19 |
| dhx_m | Reapazor: can you also try with $show_detailed_errors = ON in your config file? | 14:20 |
| Reapazor | oh i tried the dev release on the site as well | 14:21 |
| Reapazor | no it did not work | 14:21 |
| Reapazor | and i have show detailed erros on :) | 14:21 |
| Reapazor | tho do the detailed errors pass back thru soap? | 14:22 |
| dhx_m | just a blank output? | 14:22 |
| dhx_m | actually they don't... so ignore that request :) | 14:22 |
| Reapazor | why oh why wont aptana search thru files ... | 14:24 |
| Reapazor | mc_projects_get_user_accessible | 14:24 |
| Reapazor | gotta find where that is | 14:24 |
| Reapazor | thats the last function listed before the exception | 14:24 |
| dhx_m | [Mon Jun 22 13:24:13 2009] [error] [client 127.0.0.1] PHP Fatal error: Cannot use object of ty | 14:27 |
| dhx_m | pe BugData as array in /srv/www/htdocs/users/robert/mantis12x/api/soap/mc_filter_api. | 14:27 |
| dhx_m | php on line 159 | 14:27 |
| nuclear_eclipse | Reapazor: bug 10612 is reported against that function | 14:27 |
| mantisbot | Bug 10612 - rombert - open - assigned | 14:27 |
| mantisbot | mc_project_get_issue_headers returns an empty response (this bugtracker) - http://www.mantisbt.org/bugs/view.php?id=10612 | 14:27 |
| dhx_m | was that the exception you reported? | 14:27 |
| dhx_m | ah I see the problem | 14:28 |
| Reapazor | i dont know if its exactly tht | 14:29 |
| Reapazor | ill paste the exception from .net | 14:29 |
| Reapazor | sec | 14:29 |
| dhx_m | filter_api.php:1986 returns the result of bug_row_to_object | 14:29 |
| dhx_m | so we're actually returning an array of bug objects now | 14:29 |
| dhx_m | from what I gather | 14:29 |
| Reapazor | http://pastebin.ca/1469897 | 14:29 |
| dhx_m | whereas in 1.1 we would of been returning a normal array from the database | 14:29 |
| Reapazor | ahhh.... | 14:29 |
| dhx_m | and it seems not all part of Mantis have been updated to work with the new bug objects | 14:30 |
| nuclear_eclipse | paul_ said he'd be investigating the issue, so hopefully we'll be seeing a fix before I roll the rc release | 14:31 |
| dhx_m | although you must be experiencing another problem if it also occurs on 1.2.0a3? | 14:32 |
| Reapazor | will it change the outward facing functions | 14:32 |
| dhx_m | the conversion to use objects was made on 2009-06-11 | 14:32 |
| Reapazor | ala will the code im using need changes ? | 14:32 |
| nuclear_eclipse | shouldn't, afaik | 14:32 |
| dhx_m | nope | 14:32 |
| dhx_m | I might even be able to fix it... shouldn't be hard | 14:32 |
| nuclear_eclipse | but monitor bug 10612, and see what paul fixes | 14:32 |
| mantisbot | Bug 10612 - rombert - open - assigned | 14:32 |
| mantisbot | mc_project_get_issue_headers returns an empty response (this bugtracker) - http://www.mantisbt.org/bugs/view.php?id=10612 | 14:32 |
| dhx_m | just use bug->variable | 14:33 |
| dhx_m | instead of bug['variable'] | 14:33 |
| dhx_m | Reapazor: in fact, can you please try that... at mc_filter_api.php:159 | 14:33 |
| dhx_m | please change $t_id = $t_issue_data['id']; | 14:33 |
| dhx_m | to | 14:33 |
| dhx_m | $t_id = $t_issue_data->id; | 14:33 |
| Reapazor | sure second ;) | 14:35 |
| dhx_m | then check your PHP error logs on the server to see what the next exception is | 14:35 |
| dhx_m | hopefully it is mc_filter_api.php:164 | 14:36 |
| dhx_m | I'm not too sure how to test SOAP... otherwise I'd test myself | 14:36 |
| Reapazor | well i could give you a login to a test page | 14:37 |
| Reapazor | if that helps ya ? | 14:37 |
| dhx_m | really need to see PHP error logs | 14:38 |
| dhx_m | is there some sort of command line utility I'm meant to use for SOAP? | 14:38 |
| Reapazor | naw ;/ | 14:39 |
| Reapazor | ironically easiest way is to use .net :) | 14:39 |
| Reapazor | so its not generating an error | 14:39 |
| Reapazor | for php at all | 14:39 |
| Reapazor | its just coming back with access denied still | 14:39 |
| Reapazor | so maybe i should say this ... | 14:39 |
| Reapazor | the user is "api" | 14:39 |
| Reapazor | he has a nice long passwor | 14:39 |
| Reapazor | d | 14:39 |
| Reapazor | i've logged in with him to make sure he can | 14:39 |
| Reapazor | ... any other things i should have setup ? | 14:39 |
| dhx_m | I think you might have a different problem then | 14:40 |
| dhx_m | if you're getting access denied | 14:40 |
| Reapazor | http://pastebin.ca/1469897 | 14:40 |
| dhx_m | are you sure you're sending the username and password correctly? | 14:40 |
| Reapazor | thats the stack return | 14:40 |
| Reapazor | (its a commercial extension for a wiki we use) | 14:40 |
| Reapazor | it worked in prior versions | 14:40 |
| Reapazor | ala 1.1 wasnt a problem | 14:40 |
| Reapazor | but here i m wanting 'bleeding edge' | 14:41 |
| dhx_m | .NET = wall of text! :p | 14:41 |
| Reapazor | lol :) | 14:41 |
| Reapazor | here basically what the stack exception shows is i go to use that 'user accessible' function | 14:41 |
| Reapazor | and it is coming back with no | 14:41 |
| Reapazor | :) | 14:41 |
| dhx_m | I take it they don't usually have line length limits in coding standards heh | 14:41 |
| dhx_m | ah, I'm looking at a different function completely | 14:42 |
| Reapazor | hehe ya im trying to find that function | 14:42 |
| Reapazor | so i can sorta 'cheat' | 14:42 |
| Reapazor | and see whats causing it | 14:42 |
| Reapazor | but cant ;/ | 14:42 |
| Reapazor | *sigh* aptana = fail | 14:42 |
| Reapazor | i know you guys have an index online somewhere ...gonna look for that | 14:42 |
| Reapazor | found the ref :) | 14:43 |
| dhx_m | Reapazor: have you set mc_readonly_access_level_threshold and etc to the right values? | 14:43 |
| Reapazor | well the api user has adminstrator rights atm | 14:43 |
| Reapazor | one would imagine :) | 14:43 |
| Reapazor | http://pastebin.ca/1469921 | 14:44 |
| Reapazor | theres my config :) | 14:44 |
| dhx_m | Reapazor: we don't actually define mc_* defaults in config_defaults.php | 14:45 |
| dhx_m | so by default, an administrator user won't have access to SOAP API | 14:45 |
| dhx_m | you have to actually specify it in config_inc.php first | 14:45 |
| dhx_m | why it's not in config_defaults_inc.php I'm not sure | 14:45 |
| Reapazor | well i know i have tried him as just a viewer of the project as well | 14:46 |
| nuclear_eclipse | dhx_m: that's because they're specified in api/soap/mc_config_defaults_inc.php :P | 14:46 |
| dhx_m | ah :p | 14:46 |
| Reapazor | if thats the case... | 14:47 |
| dhx_m | and we even have our own config_get in there too :o | 14:47 |
| nuclear_eclipse | naturally! | 14:47 |
| Reapazor | why isnt it working ... when i've adjusted the rights | 14:47 |
| Reapazor | all over the place | 14:47 |
| Reapazor | (the user has gone all over the map - from just a viewer to admin) | 14:47 |
| Reapazor | im adding some debug stuff into get_user to find out :) | 14:50 |
| Reapazor | hehe this is why i love opensource :) | 14:50 |
| Reapazor | $t_user_id = mci_check_login( $p_username, $p_password ); | 14:51 |
| Reapazor | is coming back false | 14:51 |
| dhx_m | paul_: from 10612, I think it'd be a good idea to grep the source code like: grep -Rn "\['id'\]" * | 14:51 |
| dhx_m | paul_: and make sure all uses have been updated to work with bugobjects (where applicable) | 14:52 |
| Reapazor | i ve verified that username/password are being sent correctly | 14:52 |
| dhx_m | hmm ok | 14:52 |
| dhx_m | try placing a debug output after the auth_attempt_script_login call | 14:54 |
| dhx_m | and before the return | 14:54 |
| Reapazor | working on it ;) | 14:55 |
| Reapazor | im stepping thru the mci_check_login | 14:55 |
| Reapazor | atm | 14:55 |
| dhx_m | :) | 14:55 |
| dhx_m | you only need to place checks before return statements | 14:55 |
| Reapazor | mci_check_login | 14:55 |
| Reapazor | yes :) | 14:55 |
| Reapazor | woops | 14:55 |
| Reapazor | auth_attempt_script_login( $p_username, $p_password ) | 14:55 |
| Reapazor | is returning false... | 14:55 |
| Reapazor | off to there i go :) | 14:55 |
| dhx_m | ok | 14:55 |
| dhx_m | I guess you just keep working down to the final branch that causes the problem? | 14:57 |
| dhx_m | at the moment it is down to: user doesn't exist, user is disabled, passwords don't match | 14:58 |
| CIA-18 | Mantisbt: jreese * r30debf44a82e /doc/ (ChangeLog RELEASE): Added release notes and dropped the old changelog | 14:58 |
| CIA-18 | Mantisbt: jreese * rc69753cdb4eb /core/bug_api.php: Fix: Auto-assign not working on bug submission | 14:58 |
| paul_ | dhx_m: ya, thinking about it | 14:58 |
| dhx_m | hooray for no old changelog :) | 14:58 |
| paul_ | nuclear_eclipse: mbstring seems old functionality anyway | 14:59 |
| dhx_m | paul_: do you know how to test SOAP API easily? | 14:59 |
| paul_ | sure | 14:59 |
| paul_ | with a sledgehammer | 14:59 |
| * nuclear_eclipse beats paul_ | 14:59 | |
| dhx_m | paul_: lol | 14:59 |
| paul_ | nuclear_eclipse: I meant the lib's internally | 14:59 |
| dhx_m | maybe I'll venture into the dark realms of tests/soap | 14:59 |
| nuclear_eclipse | no, I'm beating you for breaking obvious things with your bugobject update... | 14:59 |
| paul_ | like? | 15:00 |
| paul_ | I dont touch soap api | 15:00 |
| nuclear_eclipse | CIA-18> Mantisbt: jreese * rc69753cdb4eb /core/bug_api.php: Fix: Auto-assign not working on bug submission | 15:00 |
| nuclear_eclipse | hi Glenn | 15:00 |
| Reapazor | im almost there :) | 15:00 |
| paul_ | nuclear_eclipse: well that was due to http://git.mantisbt.org/?p=mantisbt.git;a=commitdiff;h=a5f7dbcd540c533546fc80533866d808d4c64f9d | 15:01 |
| nuclear_eclipse | well, my point being that when you did the giant bugobject migration, you didn't update everything... | 15:02 |
| * nuclear_eclipse scolds paulr | 15:03 | |
| paul_ | that code was added to bugapi after the bugobject migration :P | 15:03 |
| paul_ | blame git merge :P | 15:03 |
| dhx_m | tests/soap isn't very useful... it's not a blackbox test | 15:03 |
| nuclear_eclipse | I blame paulr for not fixing merge conflicts appropriately... | 15:03 |
| dhx_m | I've just got to work out this XML format stuff | 15:04 |
| nuclear_eclipse | just toss in a bunch of angle brackets, and you'll be fine :P | 15:04 |
| Reapazor | wtf | 15:04 |
| dhx_m | haha | 15:05 |
| Reapazor | its erroring on a password mismatch ... | 15:05 |
| Reapazor | the password is 100% right | 15:05 |
| Reapazor | so ... now we work with that ;) | 15:05 |
| dhx_m | check login_method in config_inc.php | 15:05 |
| Reapazor | md5... | 15:05 |
| dhx_m | before going further, can you double check the login by logging into mantis manually with that user/pass? | 15:06 |
| dhx_m | I assume you have already though | 15:06 |
| Reapazor | yes :) | 15:06 |
| Reapazor | im tripple checking right now | 15:06 |
| Reapazor | at that level what it thinks the pw is | 15:06 |
| Reapazor | :) | 15:06 |
| [KK]Kirill | John, close this bug 10614 | 15:07 |
| mantisbot | Bug 10614 - Kirill - open - new | 15:07 |
| mantisbot | assigned issue not assigned - http://www.mantisbt.org/bugs/view.php?id=10614 | 15:07 |
| Reapazor | yup can login via that ;/ | 15:07 |
| paul_ | dhx_m: I thought I make make soap something t play with for a bit | 15:07 |
| Reapazor | darn i was hoping it was that | 15:07 |
| Reapazor | ok into that function i go | 15:07 |
| dhx_m | paul_: if you figure out what XML I'm meant to be sending, let me know :p | 15:07 |
| dhx_m | paul_: otherwise I consider it someone else's problem heh | 15:07 |
| dhx_m | Reapazor: can you please pay special attention to the utf8_substr at authentication_api.php:360 | 15:09 |
| dhx_m | or one second... | 15:09 |
| dhx_m | nah that is all I can think of | 15:10 |
| dhx_m | ohh no | 15:10 |
| Reapazor | heh actually i was looking at that line | 15:10 |
| Reapazor | :) | 15:10 |
| dhx_m | authentication_api.php:410 is probably the culprit | 15:10 |
| Reapazor | well that section | 15:10 |
| Reapazor | the password itself is long | 15:10 |
| dhx_m | a utf8_substr using PASSLEN as the limit doesn't sound good? | 15:10 |
| Reapazor | but a hash is always the same | 15:11 |
| Reapazor | working to get there ;) | 15:11 |
| dhx_m | yeah | 15:11 |
| Reapazor | soon i will have your answer | 15:11 |
| * nuclear_eclipse hates the current auth_api .... | 15:11 | |
| dhx_m | I love bugobjects :p | 15:11 |
| dhx_m | just need to expand it IMO | 15:12 |
| dhx_m | userobjects sounds nice :) | 15:12 |
| dhx_m | get rid of all the 10000000000 user_ functions | 15:12 |
| dhx_m | user_get_* in particular | 15:12 |
| dhx_m | nuclear_eclipse: is PASSLEN the number of bytes or number of characters? | 15:13 |
| nuclear_eclipse | number of characters | 15:13 |
| dhx_m | ah that's ok | 15:14 |
| nuclear_eclipse | it's the varchar() length | 15:14 |
| dhx_m | yep | 15:14 |
| dhx_m | UTF8 safe then | 15:14 |
| dhx_m | ... maybe? | 15:14 |
| nuclear_eclipse | afaik, yes | 15:14 |
| Reapazor | HTTP status 302: Found | 15:15 |
| Reapazor | its reporting that atm ... | 15:15 |
| Reapazor | *sigh* | 15:15 |
| Reapazor | still working on it :) | 15:15 |
| Reapazor | lol | 15:15 |
| dhx_m | Reapazor: check the output of the md5 function at authentication_api.php:400 | 15:15 |
| dhx_m | Reapazor: and compare it to the output of utf8_substr( $t_processed_password, 0, PASSLEN ) | 15:16 |
| dhx_m | if you can | 15:16 |
| Reapazor | hmm interesting ... | 15:16 |
| Reapazor | ya it is definately the auth part in there | 15:16 |
| Reapazor | i just was able to get it to return 'stuff' without an errror by avoiding that function | 15:16 |
| Reapazor | so ... back in i go :) | 15:16 |
| dhx_m | :) | 15:16 |
| Reapazor | :( | 15:23 |
| Reapazor | so it definately is the password len | 15:24 |
| Reapazor | :) | 15:24 |
| Reapazor | my password is > 32 chars | 15:25 |
| Reapazor | :) | 15:25 |
| Reapazor | which its getting cut down too | 15:25 |
| dhx_m | aren't you using md5? | 15:25 |
| Reapazor | i m | 15:25 |
| dhx_m | you can still have passwords > 32 chars | 15:26 |
| dhx_m | as long as you use crypt or md5 | 15:26 |
| Reapazor | ya ... but thats all i got for possible causes | 15:26 |
| Reapazor | im dead ended right now | 15:26 |
| Reapazor | lol | 15:26 |
| Reapazor | something with that ;/ | 15:26 |
| Reapazor | im gonna 'shorten' | 15:26 |
| Reapazor | the password | 15:26 |
| Reapazor | just to see | 15:26 |
| dhx_m | what line did it end up failing at? | 15:26 |
| Reapazor | didnt fail | 15:27 |
| Reapazor | preformed as it should | 15:27 |
| Reapazor | from what i can tell | 15:27 |
| dhx_m | the check after auth_process_plain_password? | 15:27 |
| dhx_m | hmm | 15:27 |
| Reapazor | ya it wasnt coming back authed | 15:27 |
| dhx_m | I'd expect it work OK from what I can see | 15:27 |
| dhx_m | (the password checking) | 15:27 |
| Reapazor | ya ;/ | 15:29 |
| Reapazor | shorter password didnt work | 15:29 |
| Reapazor | *sigh* | 15:29 |
| Reapazor | i checked the two functions as well | 15:29 |
| Reapazor | the utf8 matches the md5 | 15:29 |
| Reapazor | wait what ... | 15:30 |
| Reapazor | hold on a second here | 15:30 |
| Reapazor | this doesnt makesense | 15:30 |
| nuclear_eclipse | is soap api perhaps expecting your app to pre-md5 the password? | 15:30 |
| Reapazor | thats what im thinking :) | 15:30 |
| Reapazor | cause hte password is plaintext | 15:31 |
| Reapazor | and line 350 | 15:31 |
| Reapazor | is comparing the hash == plaintext | 15:31 |
| Reapazor | (authentification_api.php | 15:31 |
| Reapazor | looks like we've found the problem :) | 15:31 |
| Reapazor | so ..the question is ...should the soap apps be hashing already | 15:32 |
| Reapazor | ? | 15:32 |
| Reapazor | id wager against that | 15:32 |
| Reapazor | but thats not a problem | 15:32 |
| Reapazor | if it has to be :) | 15:32 |
| Reapazor | (im testing the theory right now) | 15:32 |
| nuclear_eclipse | I personally think the soap client should pre-hash the password for security reasons | 15:33 |
| nuclear_eclipse | but then again, if you're just sending the hash, it's not much different :P | 15:33 |
| nuclear_eclipse | and when we get to the point of having pluggable authentication methods, I'm not sure how that should or should not affect soap api, etc | 15:34 |
| dhx_m | nope, no security to be had unless you're using challenge-response | 15:34 |
| kenguest | nuclear_eclipse: even better: use https rather than http | 15:34 |
| dhx_m | yep | 15:34 |
| nuclear_eclipse | well, right | 15:34 |
| dhx_m | it should ideally use https with client certs | 15:34 |
| nuclear_eclipse | ssl is always the answer for web security :P | 15:34 |
| kenguest | no. | 15:35 |
| kenguest | but it helps | 15:35 |
| dhx_m | I guess just server https is also OK | 15:35 |
| kleinweby | (because you're currently talking about soap): Is there a way to "searchs" with the maintis soap api like "All assigned to me"? | 15:35 |
| kleinweby | mantis, sorry | 15:36 |
| dhx_m | not too sure | 15:37 |
| nuclear_eclipse | ironically, I think the only devs that actually use the soap api aren't here :P | 15:37 |
| dhx_m | lol | 15:37 |
| kleinweby | oO | 15:38 |
| Reapazor | bad news | 15:39 |
| Reapazor | still no dice with hash being sent | 15:39 |
| Reapazor | sigh ;/ | 15:39 |
| Reapazor | ill keep going ;/ | 15:39 |
| Reapazor | now with the hash thing i gotta start at the top | 15:41 |
| Reapazor | :) good thing im not expected to do anything today lol | 15:41 |
| dhx_m | lol | 15:41 |
| Reapazor | *sigh* still no luck | 16:03 |
| Reapazor | i just cant win | 16:03 |
| Reapazor | LOL | 16:03 |
| Reapazor | # Do not support migration to PLAIN, since this would be a crazy thing to do. | 16:19 |
| Reapazor | # Also if we do, then a user will be able to login by providing the MD5 value | 16:19 |
| Reapazor | # that is copied from the database. See #8467 for more details. | 16:19 |
| Reapazor | if( $t_configured_login_method != PLAIN && $t_login_method == PLAIN ) { | 16:19 |
| Reapazor | return "should be true"; | 16:19 |
| Reapazor | continue; | 16:19 |
| Reapazor | } | 16:19 |
| Reapazor | (yes thats my return) | 16:19 |
| Reapazor | thats my problem | 16:19 |
| Reapazor | auth_does_password_match | 16:19 |
| Reapazor | so you 'want' md5 | 16:19 |
| Reapazor | but then when it gets there | 16:19 |
| Reapazor | it doesnt know whats going on | 16:19 |
| Reapazor | there my problem... | 16:20 |
| Reapazor | but now how to resolve that | 16:20 |
| Reapazor | lol | 16:20 |
| Reapazor | oh and the user_get_access_level also doesnt respect global access levels it seems ;) | 16:30 |
| paul_ | nuclear_eclipse: oh john | 16:34 |
| paul_ | that reminds me | 16:34 |
| paul_ | did we talk about mssql? :) | 16:34 |
| * paul_ forgets | 16:34 | |
| paul_ | Reapazor: I dislike the current soap api a little | 16:36 |
| Reapazor | meh .. i still havent got it workiing ;/ | 16:37 |
| Reapazor | i can see one global issue ... | 16:37 |
| Reapazor | im passing user/password to login... | 16:37 |
| Reapazor | but post that its treating me as anoynmous | 16:37 |
| Reapazor | ala when it gets access levels | 16:37 |
| Reapazor | its pulling for my anonymous user | 16:37 |
| paul_ | nuclear_eclipse: can we branch? | 16:41 |
| Reapazor | Mon Jun 22 12:42:03 2009] [error] [client 70.32.66.194] PHP Fatal error: Cannot use object of type BugData as array in /var/www/vhosts/dotbunny.com/subdomains/bugs/httpdocs/api/soap/mc_project_api.php on line 31 | 16:42 |
| Reapazor | yay! a core issue ! | 16:43 |
| Reapazor | no more auth issues back to the bugdata | 16:43 |
| paul_ | cool | 16:43 |
| paul_ | what was it? | 16:43 |
| Reapazor | б═б═б═ б═б═б═ б═б═б═if( $t_configured_login_method != PLAIN && $t_login_method == PLAIN ) { | 16:44 |
| Reapazor | б═б═б═ б═б═б═ б═б═б═ б═б═б═continue; | 16:44 |
| Reapazor | б═б═б═ б═б═б═ б═б═б═} | 16:44 |
| Reapazor | sec getting line number | 16:44 |
| Reapazor | its just the auth part ... | 16:44 |
| Reapazor | the bigger issue is my logged in user isnt having his access level rightly pulled | 16:44 |
| Reapazor | ;/ | 16:44 |
| Reapazor | but in the end boils down to that bugdata object error | 16:45 |
| Reapazor | after the auth stuff ... | 16:46 |
| nuclear_eclipse | paul_: we haven't talked about mssql afiak, but I'd like to wait for you to look at the soap stuff before we branch | 16:53 |
| dhx_m | nuclear_eclipse: is RELEASE meant to list as many "main" features as possible? | 17:20 |
| dhx_m | guess not :) | 17:21 |
| dhx_m | the changelog would be far too large now that I look at changelog_page.php | 17:22 |
| paul_ | nuclear_eclipse: branch then port! | 17:31 |
| nuclear_eclipse | paul_: really, have patience | 17:33 |
| nuclear_eclipse | what's so important that you can't keep it in a private branch for a couple more days at the most? | 17:33 |
| paul_ | we've done that before :P | 17:33 |
| nuclear_eclipse | and if you really want 1.2 branched so bad, fix the soap api stuff from bug 10612 and so forth, and then I'll branch | 17:33 |
| mantisbot | Bug 10612 - rombert - open - assigned | 17:33 |
| mantisbot | mc_project_get_issue_headers returns an empty response (this bugtracker) - http://www.mantisbt.org/bugs/view.php?id=10612 | 17:33 |
| paul_ | you know I need to do more then that | 17:34 |
| paul_ | iirc, there's 27 | 17:34 |
| paul_ | well 28 now | 17:34 |
| paul_ | soap api bugs to review | 17:34 |
| paul_ | of which 4 contain patchsets | 17:34 |
| * paul_ sets to work | 17:34 | |
| * nuclear_eclipse didn't realize soap api was that broken... | 17:35 | |
| paul_ | no but there's lots of stuff missing | 17:36 |
| paul_ | also need to fix the utf8 bugs that seem to be coming up now | 17:36 |
| dhx_m | yep I'd like some more testing before RC | 17:38 |
| paul_ | huh | 17:38 |
| paul_ | testing? | 17:38 |
| dhx_m | RC is meant to be "this should be stable" | 17:38 |
| dhx_m | the bugs we're finding aren't just minor things | 17:38 |
| dhx_m | although | 17:38 |
| dhx_m | I think branching is good for people that want to start working on new features again :) | 17:38 |
| nuclear_eclipse | RC imo means "representative of a final release" | 17:39 |
| dhx_m | ... but I imagine we're all focussing on fixing at the moment | 17:39 |
| dhx_m | that too | 17:39 |
| nuclear_eclipse | it doesn't need to be completely bug free, but it should be relatively ready for primetime use | 17:39 |
| dhx_m | but users are going to hate us if we release an RC where the most obvious things break | 17:39 |
| dhx_m | yep | 17:39 |
| nuclear_eclipse | at the same time, I'd rather have an RC release sooner where a bunch of users can submit bug reports than let it sit in Git forever while we don't get it out into the feild enough to even find those issues | 17:40 |
| dhx_m | yep | 17:42 |
| dhx_m | paul_: do you have a link to the test site you showed me ages ago? | 17:42 |
| dhx_m | paul_: I really like that idea :) | 17:42 |
| paul_ | hmm? | 17:42 |
| dhx_m | paul_: the unit test stuff | 17:43 |
| paul_ | you mean http://www.mantisforge.org/cruisecontrol/buildresults/mantisbt?tab=documentation | 17:43 |
| paul_ | or well | 17:43 |
| paul_ | http://www.mantisforge.org/cruisecontrol/buildresults/mantisbt?tab=testResults | 17:43 |
| paul_ | for some reason | 17:43 |
| paul_ | http://www.mantisforge.org/cruisecontrol/buildresults/mantisbt?tab=phpcs isn't showing output | 17:43 |
| dhx_m | yep | 17:43 |
| * paul_ needs to fix codesniffer output :( | 17:45 | |
| dhx_m | now I just need to learn how to add tests | 17:45 |
| paul_ | make the config api test thing into a test | 17:45 |
| paul_ | if you want something to do :) | 17:45 |
| dhx_m | do the tests run on a live database? | 17:45 |
| dhx_m | ie. it actually inserts and deletes from the database? | 17:45 |
| paul_ | they would do | 17:46 |
| paul_ | atm | 17:46 |
| dhx_m | ok :) | 17:47 |
| dhx_m | oh btw, did anyone see the deprecation notices in database_api.php? | 17:52 |
| dhx_m | can we either remove the version if not fixing in 1.2 - or remove the deprecated stuff? | 17:52 |
| paul_ | I swwear I already fixed this file | 17:54 |
| nuclear_eclipse | paul_: have you seen bug 9798? | 18:01 |
| mantisbot | Bug 9798 - bkraul - open - feedback | 18:01 |
| mantisbot | Unable to perform Mantis upgrade from 1.1.2 to 1.2.0a2 MSSQL(SQL 2005 Express) - http://www.mantisbt.org/bugs/view.php?id=9798 | 18:01 |
| paul_ | Reapazor: stiill here? | 18:02 |
| Reapazor | yes | 18:04 |
| Reapazor | whats shaking? | 18:04 |
| paul_ | try pulling trunk and testing your thingy | 18:04 |
| Reapazor | okie give me a min | 18:05 |
| Reapazor | fixed the auth part ? | 18:05 |
| paul_ | no the filter part | 18:05 |
| CIA-18 | Mantisbt: paul * r76dbbcfb8307 /api/soap/mc_filter_api.php: #0010612: mc_project_get_issue_headers returns an empty response (this bugtracker) | 18:05 |
| Reapazor | ok lemme see what i can do :) | 18:06 |
| paul_ | nuclear_eclipse: well, I think we need to update install.php to cover what we actually support | 18:07 |
| * paul_ uses mssql with odbc | 18:08 | |
| nuclear_eclipse | what does install.php have to do with bad SQL syntax or what not during the schema install/upgrade? | 18:08 |
| paul_ | there's multiple mssql drivers | 18:08 |
| paul_ | i.e. | 18:09 |
| paul_ | you can use | 18:09 |
| paul_ | mssql | 18:09 |
| paul_ | ado_mssql | 18:09 |
| paul_ | odbc_mssql | 18:09 |
| paul_ | pdo_mssql | 18:09 |
| * paul_ thinks | 18:09 | |
| paul_ | the MS native mssql driver | 18:09 |
| Reapazor | http://pastebin.com/m4f6c2ef4 | 18:09 |
| nuclear_eclipse | oh, so you think that bug is a driver problem, not a syntax problem? | 18:09 |
| Reapazor | now i get someerrors back :) | 18:09 |
| paul_ | install.php uses 'mssql' | 18:09 |
| paul_ | I tend to use odbc_mssql at work | 18:09 |
| paul_ | (as iirc, mssql truncates strings at the old sybase 8000 char limit | 18:09 |
| paul_ | and MS have now done there new driver | 18:10 |
| paul_ | which is version 1.0 | 18:10 |
| paul_ | but won't be useful for us until version 1.1 gets released | 18:10 |
| paul_ | On top of that mess | 18:10 |
| paul_ | sql server stores utf8 data as ucs-2? (or similar typo) | 18:10 |
| paul_ | in 1.1 *apparently* they going to support some sort of translation to/from utf8 | 18:11 |
| paul_ | I'm not quite sure atm which of ado/odbc etc support utf8... | 18:11 |
| nuclear_eclipse | so, basically MSSQL is crap :P | 18:12 |
| paul_ | on top of that | 18:13 |
| paul_ | I made the mistake of asking about lowercase'ing some specific utf8 chars | 18:13 |
| paul_ | in php | 18:13 |
| dhx_m | nuclear_eclipse: yeah... UCS-2 doesn't support surrogates and hence doesn't support the full range of valid Unicode characters :o | 18:14 |
| nuclear_eclipse | whatever the hell that all means :P | 18:14 |
| paul_ | It seems that mbstring doesn't use the same utf8 libs that the php6 uses | 18:14 |
| paul_ | (or php5.3) | 18:14 |
| paul_ | with it's intl extension | 18:14 |
| paul_ | dhx_m: whats a surrogate? | 18:14 |
| paul_ | isn't that a type of pregnancy? | 18:15 |
| dhx_m | paul_: it is a second 16bit value that follows an initial 16bit value... this allows more characters to be represented | 18:16 |
| dhx_m | paul_: basically with UCS-2 they decided "oh wait, 65536 characters is not enough to fit everything" | 18:16 |
| nuclear_eclipse | facepalm | 18:16 |
| dhx_m | paul_: and then they came up with UTF-16 which is quite similar to how UTF-8 works... except the base unit is a 16bit character (not 8bit) | 18:17 |
| nuclear_eclipse | damn all those people and their pictograms and such :P | 18:17 |
| dhx_m | :) | 18:17 |
| dhx_m | UTF8 is great because you can just keep extending it as much as you like | 18:17 |
| paul_ | anyway | 18:17 |
| nuclear_eclipse | if everyone would just learn ascii and english, programs would be so much easier to write! | 18:17 |
| Reapazor | paul_: http://pastebin.com/m4f6c2ef4 ... ;/ | 18:17 |
| dhx_m | well maybe not as much as you like... but anyway :) | 18:17 |
| dhx_m | haha | 18:17 |
| paul_ | i've taking unicode data | 18:17 |
| paul_ | from unicode.org | 18:18 |
| paul_ | run it over mbstring/compatlibs and so forth | 18:18 |
| dhx_m | just grab it from zh.wikipedia.org and similar :p | 18:18 |
| paul_ | and found that all the libraries I have (include phputf8 amongst others) | 18:18 |
| paul_ | vary to mbstring by a bunch of characters | 18:18 |
| paul_ | and i'm now trying to work out if mbstring or the libs are wrong | 18:19 |
| dhx_m | did you set the locale correctly? | 18:19 |
| paul_ | iirc | 18:19 |
| paul_ | according to mbstring | 18:19 |
| dhx_m | those utf8 functions have different outputs depending on the locale | 18:19 |
| paul_ | LATIN UPPERCAZE Z IN A CIRCLE | 18:19 |
| paul_ | stays as LATIN UPPERCAZE Z IN A CIRCLE when lowercase'd iirc | 18:19 |
| dhx_m | for instance, "a" might be lowercase in English but it might be uppercase in WeirdLanguage | 18:20 |
| paul_ | the unicode data suggests it should become latin lowercase z in a circle | 18:20 |
| dhx_m | paul_: a lot of these libraries have a very limited lookup table for each language as well | 18:20 |
| paul_ | http://www.fileformat.info/info/unicode/char/24cf/index.htm | 18:21 |
| paul_ | http://unicode.org/cldr/utility/character.jsp?a=24C9 | 18:23 |
| paul_ | see what mb_strtolower() of char 24C9 returns for you | 18:23 |
| paul_ | Reapazor: mc_api.php | 18:25 |
| paul_ | line 196 | 18:25 |
| paul_ | Reapazor: try removing the & from function mci_null_if_empty( &$p_value ) { | 18:25 |
| Reapazor | sec :) | 18:26 |
| Reapazor | holy crap 1/3 tests passed!! | 18:27 |
| paul_ | 1/3? | 18:27 |
| Reapazor | hehe ya :) i am testing different api function calls | 18:27 |
| Reapazor | the other ones are coming back with another .... message | 18:28 |
| Reapazor | something about content type being returned | 18:28 |
| Reapazor | not expected | 18:28 |
| paul_ | function mci_get_severity_id( &$p_severity ) { | 18:28 |
| paul_ | i'm tempted to remove all the &'s | 18:28 |
| * paul_ pokes nuclear_eclipse | 18:28 | |
| dhx_m | passing by reference is more efficient though? | 18:28 |
| nuclear_eclipse | hi paul_ | 18:29 |
| Reapazor | im getting some Not supported Content-Type in the response: 'text/html' | 18:29 |
| Reapazor | errors ;/ | 18:29 |
| Reapazor | as well as System.Web.Services.Protocols.SoapException: Invalid Filter | 18:30 |
| Reapazor | now :) | 18:30 |
| nuclear_eclipse | timezone chooser breaks on mantisbt.org because the timezone_identifiers_list() function is not in 5.1 as a core function | 18:30 |
| paul_ | (PHP 5 >= 5.1.0) | 18:30 |
| paul_ | Installation | 18:30 |
| paul_ | There is no installation needed to use these functions; they are part of the PHP core. | 18:30 |
| nuclear_eclipse | not to mention I want to stab the list that you're having it use anyways because it's a) horribly verbose, and b) not specific | 18:31 |
| nuclear_eclipse | read the first comment: "Please note that the timezone_identifiers_list function is not available in the most recent versions of PHP available for CentOS/RHEL as of this writing ( 5.1.6-23.2.el5_3 )." | 18:31 |
| Reapazor | ^ that could be me ? | 18:31 |
| paul_ | ... | 18:31 |
| paul_ | nuclear_eclipse: why not? ;/ | 18:31 |
| nuclear_eclipse | "I think the function is labeled as >= 5.1.0 in this documentation because it's possible to get the Date::Time class installed in 5.1 if you're compiling from scratch (or it's a documentation error). | 18:32 |
| nuclear_eclipse | "I just spent a bunch of time trying to get it to work in 5.1 (installing timezonedb manually because the default PECL runs out of memory, etc.), but have decided to use a manually generated list instead. The actual timezone support seems to work fine in 5.1." | 18:32 |
| dhx_m | paul_: you'll soon learn that the sole aim of RHEL is to frustrate sysadmins :p | 18:32 |
| paul_ | nuclear_eclipse: right, let me find out | 18:32 |
| paul_ | i'll ask the php-dev who updates the timezone db | 18:33 |
| paul_ | whether RH fail | 18:33 |
| nuclear_eclipse | paul_: I've confirmed via command line on the server that the funciton does not exist in our installation | 18:33 |
| nuclear_eclipse | anyways, I think that list is disingenuous anyways for the reasons I stated above -- we should present a much smaller list to the user, including +/- from GMT | 18:34 |
| paul_ | I'd agree there | 18:34 |
| paul_ | but I also think we should try to use the php functions to do the work for us | 18:34 |
| paul_ | (They have a more accurate timezone db) | 18:34 |
| nuclear_eclipse | +1, but it doesn't seem to have anything like that | 18:35 |
| paul_ | whilst we need a simple list | 18:35 |
| paul_ | you can get the timezone stuff as an object and do whatever we want to | 18:35 |
| paul_ | dinner brb | 18:35 |
| dhx_m | using +/- hours from GMT disregards daylight savings time though | 18:39 |
| dhx_m | which is wrong | 18:39 |
| nuclear_eclipse | gah, let's please not bring DST into this mess just yet :P | 18:40 |
| dhx_m | :p | 18:41 |
| paul_ | well we first need to work out why RHEL doesn't have the function | 18:49 |
| paul_ | as i'd rather use the php timezone stuff | 18:49 |
| paul_ | then roll our own database | 18:49 |
| paul_ | (especially given our names need to match theres anyway) | 18:49 |
| Reapazor | paul_ ...seems the only problem i have left is mc_project_api.php:line 54 | 18:50 |
| Reapazor | PHP Fatal error: Cannot use object of type BugData as array in /var/www/vhosts/dotbunny.com/subdomains/bugs/httpdocs/api/soap/mc_project_api.php on line 54 | 18:50 |
| dhx_m | I guess a fallback to using +/- hrs from GMT would be acceptable | 18:50 |
| dhx_m | but too much work IMO | 18:50 |
| paul_ | Reapazor: did you remove & to all ? | 18:51 |
| Reapazor | not in that file yet | 18:51 |
| Reapazor | in the other :) | 18:51 |
| Reapazor | hmm looks like i might have missed one | 18:52 |
| Reapazor | *sigh* back i go | 18:52 |
| Reapazor | ah different file :) | 18:52 |
| Reapazor | mc_enum_api | 18:52 |
| Reapazor | as i go thru you seem to be right on | 18:56 |
| Reapazor | all the &'s are breaking it | 18:56 |
| paul_ | nuclear_eclipse: *sigh* ok having googled | 18:57 |
| paul_ | there's 2 things here | 18:57 |
| paul_ | a) it seems the datetime stuff was experimental in php 5.1 | 18:57 |
| paul_ | and default from 5.2.0 | 18:58 |
| paul_ | seperately to that | 18:58 |
| Reapazor | line 54 of mc_project_api .... was $t_issue_data['handler_id'] needs to be $t_issue_data->handler_id | 18:58 |
| Reapazor | that puts me up to 3 of 4 tests passed ! | 18:58 |
| paul_ | there's some whinging about php distributing timezones (to fix issues with distributions updating and breaking the timezonedb on applications behind the scene) | 18:58 |
| Reapazor | ln 30 in mc_project_api should read: $t_id = $t_issue_data->id; | 18:59 |
| Reapazor | assuming someone did that, or thats left from my fix | 18:59 |
| paul_ | my original plan with that drop down (Timezones) | 18:59 |
| paul_ | was to improve it later on to put common ones at top of list or have a short list | 19:00 |
| paul_ | then have a long list listing everything to pick up the 'random timezones' people might want to use | 19:00 |
| nuclear_eclipse | I think just having a short list would be the best option | 19:00 |
| paul_ | I was just trying to avoid the case of people 'bug-reporting' | 19:01 |
| paul_ | Please add Europe/Athens to timezone | 19:01 |
| paul_ | as on one hand, the complete list that people need to use is at http://us.php.net/manual/en/timezones.php | 19:01 |
| paul_ | if a Server is set to | 19:01 |
| paul_ | Europe/Athens | 19:01 |
| paul_ | but that isn't on our pre-defined dropdown list | 19:02 |
| paul_ | we'd have to add it or something | 19:02 |
| nuclear_eclipse | if we don't really categorize timezones by their "city", then we shouldn't get those sorts of bug reports, no? | 19:02 |
| Reapazor | so where might i go to tell / write down the changes ? | 19:02 |
| Reapazor | as 4/5 test work | 19:02 |
| paul_ | *maybe* | 19:02 |
| Reapazor | (i keep adding a few) | 19:03 |
| Reapazor | 5'th being a filter error | 19:03 |
| nuclear_eclipse | Reapazor: post them to a bug report on mantisbt.org please | 19:03 |
| * paul_ slaps nuclear_eclipse | 19:03 | |
| paul_ | nuclear_eclipse: the point is | 19:03 |
| paul_ | a) we need to use a valid php timezone | 19:03 |
| paul_ | b) users could use any valid phptimezone in their php.ini | 19:03 |
| paul_ | so we'd effectively be saying | 19:03 |
| nuclear_eclipse | we can't just map all the possible +/- settings to a single timezone? | 19:04 |
| nuclear_eclipse | it's not like London will all of a sudden move over +3 hours... | 19:04 |
| paul_ | I was wondering about summer time in places but not others or similar | 19:04 |
| nuclear_eclipse | screw DST? | 19:05 |
| nuclear_eclipse | :P | 19:05 |
| paul_ | the db stores timezone offsets http://us.php.net/manual/en/datetimezone.getoffset.php | 19:05 |
| paul_ | the only thing we'd need to support is: | 19:06 |
| paul_ | if user sets timezone in php.ini as 'Australia/Lord_Howe' | 19:06 |
| paul_ | that would be default for people that haven't set a timezone (aka server timezone) | 19:06 |
| paul_ | so whilst I knew it was a long list | 19:06 |
| paul_ | i'd be kinda hoping we could 'tidy' it as time goes by ;/ | 19:07 |
| paul_ | In this case it doesn't help that | 19:07 |
| paul_ | php manual is wrong | 19:07 |
| paul_ | my windows boxes run php5.3 and my linux boes run 5.2 | 19:07 |
| paul_ | (linux box runs stock ubuntu stuff) | 19:07 |
| paul_ | looks like wordpress have the same issue http://wordpress.org/support/topic/279593 | 19:09 |
| paul_ | albeit it looks like the solution they have is not to allow users to change timezone unless they are on 5.2 | 19:09 |
| nuclear_eclipse | hmm | 19:10 |
| dhx_m | preg_replace() [function.preg-replace]: Compilation failed: PCRE does not support \L, \l, \N, \P, \p, \U, \u, or \X at offset 98 | 19:10 |
| paul_ | the http://gophp5.org/ date was like Feb 2008 | 19:10 |
| paul_ | and that was saying 'go php5.2 as 5.0 was crap and 5.1 was slow' | 19:11 |
| nuclear_eclipse | dhx_m: did you use the /u modifier+ | 19:11 |
| dhx_m | string_api.php:473 | 19:11 |
| dhx_m | nuclear_eclipse: we already are | 19:11 |
| paul_ | what are you changing ? | 19:11 |
| mantisbot | New bug: Bug 10615 - reapazor - open - new | 19:11 |
| mantisbot | New bug: API Failure - http://www.mantisbt.org/bugs/view.php?id=10615 | 19:11 |
| nuclear_eclipse | paul, look at 10615 and commit the fixes | 19:12 |
| dhx_m | nuclear_eclipse: mind you, this is a warning... I have all values for $g_display_errors set to 'halt' | 19:12 |
| paul_ | dhx_m: we shouldn't be getting any warnings | 19:13 |
| paul_ | nuclear_eclipse: already have | 19:13 |
| dhx_m | nuclear_eclipse: using CentOS 4 with custom upgraded packages | 19:13 |
| paul_ | or well | 19:13 |
| paul_ | i've fixed a ocuple more | 19:13 |
| CIA-18 | Mantisbt: paul * r996615c7f346 /api/soap/ (mc_api.php mc_project_api.php): Fix additional soap api issues | 19:13 |
| paul_ | Reapazor1: did i miss anything? | 19:14 |
| dhx_m | paul_: the great folks at RHEL/CentOS must have decided no one needs Unicode support in PCRE | 19:14 |
| paul_ | hmm? | 19:14 |
| paul_ | isn't it /u? | 19:14 |
| dhx_m | yep | 19:14 |
| nuclear_eclipse | dhx_m: upgrade to at *least* CentOS 5 :P | 19:15 |
| dhx_m | nuclear_eclipse: I can't :( | 19:15 |
| paul_ | ? | 19:15 |
| dhx_m | this isn't my server | 19:15 |
| nuclear_eclipse | bad dhx_m | 19:15 |
| nuclear_eclipse | no biscuit! | 19:15 |
| paul_ | dhx_m: are you on php5.1? | 19:16 |
| paul_ | on that box? | 19:16 |
| Reapazor1 | paul_ ill do a trunk pull | 19:16 |
| Reapazor1 | and see how she works | 19:16 |
| paul_ | mantis is male | 19:16 |
| Reapazor1 | lol fine ... | 19:16 |
| dhx_m | paul_: yes, 5.1.6 | 19:16 |
| Reapazor1 | ill see how he works | 19:16 |
| paul_ | dhx_m: what pcre version? | 19:16 |
| paul_ | in phpinfo? | 19:16 |
| paul_ | dhx_m: let me guess | 19:17 |
| paul_ | 4.5 | 19:17 |
| dhx_m | paul_: 4.5 01-December-2003 | 19:17 |
| dhx_m | yep | 19:17 |
| paul_ | wget php5.3 source | 19:17 |
| paul_ | ./configure | 19:18 |
| paul_ | make | 19:18 |
| paul_ | make install | 19:18 |
| paul_ | and move on with life | 19:18 |
| dhx_m | paul_: I wish I could... it's a shared server I have no control over | 19:18 |
| Reapazor1 | paul_ seems to be working :) | 19:18 |
| Reapazor1 | now there is one 'problem' that i dont know if i really care to address | 19:18 |
| paul_ | Reapazor1: I guess your not on php5.1 then | 19:18 |
| paul_ | :P | 19:19 |
| Reapazor1 | LOL | 19:19 |
| Reapazor1 | the filters | 19:19 |
| Reapazor1 | they changed in 1.2 ? | 19:19 |
| paul_ | unknown | 19:19 |
| paul_ | in fact probably | 19:19 |
| paul_ | lots cfhanges | 19:19 |
| Reapazor1 | " Invalid Filter " | 19:19 |
| Reapazor1 | when i do a soap search by filter | 19:19 |
| dhx_m | paul_: A non well formed numeric value encountered | 19:20 |
| dhx_m | paul_: custom_function_api.php:333 | 19:20 |
| paul_ | what's php5.1 fail on now? | 19:20 |
| dhx_m | paul_: arguments to date(): ( <string>'Y-m-d', <string>'2009-06-19 23:45:43' ) | 19:21 |
| Reapazor1 | paul_ : ya 5.2.6 atm :) | 19:21 |
| paul_ | $t_function( $p_bug, $p_columns_target ); | 19:21 |
| paul_ | is custom function api 333 | 19:21 |
| dhx_m | yep | 19:22 |
| paul_ | thats not a date.... | 19:22 |
| dhx_m | sorry | 19:22 |
| dhx_m | wrong line of the trace | 19:22 |
| dhx_m | columns_api.php:1079 | 19:23 |
| dhx_m | it's grabbing $p_bug->last_updated | 19:23 |
| dhx_m | and it seems to be a string? | 19:23 |
| paul_ | what had you done to generate that ;/ | 19:25 |
| dhx_m | paul_: doh never mind me... | 19:25 |
| paul_ | ? | 19:26 |
| dhx_m | paul_: hadn't fully upgrade the database yet *headslam* | 19:26 |
| paul_ | although is tempting to add if(!is_numeric( to __Set for bug | 19:27 |
| dhx_m | yep | 19:27 |
| dhx_m | I was hoping that in the future all checks are placed in __set/__get | 19:28 |
| dhx_m | ie. who can update fields, etc | 19:28 |
| CIA-18 | Mantisbt: jreese * r6ee263adfacc / (account_prefs_update.php core/print_api.php): Fix timezone list on some PHP 5.1 installations | 19:29 |
| nuclear_eclipse | ^ temporary measures | 19:30 |
| dhx_m | nuclear_eclipse: can we check for PCRE Unicode support? | 19:34 |
| * nuclear_eclipse stabs dhx_m | 19:34 | |
| dhx_m | nuclear_eclipse: it'd be easy :p | 19:34 |
| nuclear_eclipse | talk to paul_, he's the overmind behind admin/checks.php | 19:35 |
| dhx_m | nah I meant inline | 19:35 |
| dhx_m | so... | 19:35 |
| * nuclear_eclipse stabs dhx_m | 19:35 | |
| dhx_m | $t_valid_unicode_chars = is_unicode_supported() ? '\pL' : ''; | 19:36 |
| dhx_m | something like that | 19:36 |
| dhx_m | nuclear_eclipse: actually I'm of the opinion now that old crap like CentOS 4 should be providing their own patches to revert Unicode/etc support | 19:40 |
| dhx_m | at least for small things like I'm experiencing now | 19:40 |
| nuclear_eclipse | I'm of the opinion that anyone still running CentOS 4 should be taken out back and beaten or shot | 19:41 |
| nuclear_eclipse | :P | 19:41 |
| paul_ | dhx_m: i'm thinking about your /u thing | 19:41 |
| dhx_m | nuclear_eclipse: I hate it too! | 19:41 |
| paul_ | so | 19:47 |
| paul_ | remind me | 19:47 |
| paul_ | on centos 4 | 19:47 |
| paul_ | utf8 fails? | 19:47 |
| paul_ | so we're telling people to update? :P | 19:47 |
| dhx_m | yeah | 19:48 |
| dhx_m | RHEL 5 is ok then? | 19:48 |
| dhx_m | the fix seems really simple at the moment | 19:48 |
| nuclear_eclipse | dhx_m: it seems so, mantisbt.org runs it | 19:48 |
| paul_ | RHEL seems to be as bad :P | 19:48 |
| dhx_m | but as we use more and more unicode regex queries... | 19:48 |
| nuclear_eclipse | or rather, live site runs CentOS 5 | 19:49 |
| dhx_m | yep, same thing I guess | 19:50 |
| dhx_m | if we're going to release this | 19:51 |
| dhx_m | actually never mind ;) | 19:52 |
| dhx_m | was going to say the CentOS/RHEL fix is easy | 19:52 |
| nuclear_eclipse | paul_: any other immediate fixes to soap api available/needed? | 19:52 |
| dhx_m | but it isn't really... you need a custom PHP/Apache/1000 other things too | 19:52 |
| dhx_m | nuclear_eclipse: what is the reason for http://git.mantisforge.org/w/source-integration.git?a=commitdiff;h=67994d425f8a81ec6a0767a3f4de7b930225bb81 | 19:55 |
| dhx_m | all throughout | 19:56 |
| nuclear_eclipse | because if/when the API function gets called from a different plugin, the plugin API will look in the wrong place for the configuration option due to the way the plugin stack works | 19:56 |
| dhx_m | yep... but what calls the Source plugin? | 19:56 |
| nuclear_eclipse | a) other Source-based plugins, b) the SourceIntegration sub-plugin | 19:57 |
| dhx_m | sounds like something we should fix with plugin_config_get so that it takes an optional "plugin" argument? | 19:57 |
| dhx_m | then again... may as well use the raw thing | 19:58 |
| nuclear_eclipse | it would be preferred, but there are already multiple, optional parameters already, that you'd have to call plugin_config_get( 'value', null, null, null, 'Source' ); | 19:58 |
| dhx_m | I see your point now though :) | 19:58 |
| dhx_m | ah ok | 19:59 |
| paul_ | re-order params :P | 19:59 |
| nuclear_eclipse | at which point, I figured it easier to make a call to the core API :P | 19:59 |
| nuclear_eclipse | re-order params would break anything tha'ts been relying on the API up to this point... | 19:59 |
| nuclear_eclipse | if PHP had named params like Python, I would have added it in a heartbeat | 20:00 |
| dhx_m | yep | 20:00 |
| nuclear_eclipse | eg, plugin_config_get( 'value', plugin='Source' ) type of thing | 20:00 |
| nuclear_eclipse | I <3 Python | 20:00 |
| dhx_m | yeah that is what I was thinking too (but in terms of Ada :p) | 20:00 |
| dhx_m | Undefined property: SourceChangeset::$comitter | 20:04 |
| dhx_m | Source/Source.ViewAPI.php:120 | 20:05 |
| nuclear_eclipse | hah, gotta love typos... :P | 20:06 |
| nuclear_eclipse | that one's *really* bad.... :( | 20:06 |
| dhx_m | lol | 20:07 |
| nuclear_eclipse | fix pushed | 20:07 |
| dhx_m | thanks :) | 20:08 |
| nuclear_eclipse | np | 20:10 |
| nuclear_eclipse | *that* is why I haven't yet made a real release of source-integration yet :P | 20:10 |
| dhx_m | :) | 20:11 |
| dhx_m | fixed thanks | 20:12 |
| dhx_m | paul_: with the new graphs plugin... I can't specify the path to jpgraph anymore | 20:13 |
| paul_ | you'd put jpgraph in libs dir | 20:13 |
| dhx_m | ah ok, it's forced now? | 20:13 |
| dhx_m | I don't mind that :) | 20:13 |
| dhx_m | the other ezc thing was complaining about missing fonts and I couldn't be bothered debugging :p | 20:15 |
| nuclear_eclipse | paul_: any other immediate fixes to soap api available/needed? anything to add to the release notes? | 20:16 |
| paul_ | what release notes? | 20:16 |
| nuclear_eclipse | doc/RELEASE | 20:16 |
| dhx_m | minimum system requirements need updating | 20:19 |
| CIA-18 | Mantisbt: jreese * rd11b55e8c96d /doc/INSTALL: Updated documented minimum PHP version. | 20:21 |
| dhx_m | is MySQL really a requirement? | 20:23 |
| nuclear_eclipse | yes :P | 20:23 |
| paul_ | no :P | 20:23 |
| dhx_m | hah | 20:23 |
| nuclear_eclipse | just look at the crap paul's going through for pgsql and mssql support :P | 20:24 |
| paul_ | since when did we have a docs/RELEASE | 20:24 |
| dhx_m | oh pgsql doesn't work :o | 20:24 |
| nuclear_eclipse | since this morning | 20:24 |
| dhx_m | I don't use it... but it'd be my choice for database if I was installing one anytime soon | 20:24 |
| * nuclear_eclipse notes that paul_ should really pay attention to git update emails... | 20:24 | |
| paul_ | well, hmm | 20:27 |
| dhx_m | manage_user_create.php:82 $t_lang is undefined | 20:30 |
| paul_ | mm? | 20:31 |
| nuclear_eclipse | needs to be " = lang_get_current()" | 20:32 |
| dhx_m | how did that happen? :o | 20:33 |
| dhx_m | that fixes is | 20:34 |
| dhx_m | *it | 20:34 |
| CIA-18 | Mantisbt: jreese * r6a862b67e037 /manage_user_create.php: Fix warnings about $t_lang and such. | 20:34 |
| nuclear_eclipse | ^ even better fix | 20:35 |
| dhx_m | yep | 20:36 |
| nuclear_eclipse | anywho, time to head home -- will be back on later | 20:36 |
| nuclear_eclipse | cheers | 20:36 |
| dhx_m | ok cya :) | 20:36 |
| paul_ | we miss some big chunks of work in those release notes | 20:39 |
| dhx_m | I'm more excited about what to do in 1.3 :p | 20:46 |
| paul_ | I think we should pick a date for 1.3 | 20:48 |
| paul_ | for the release | 20:48 |
| dhx_m | quarterly releases would be nice | 20:52 |
| dhx_m | maybe a bit fast though? | 20:52 |
| dhx_m | I guess we have branching now anyway | 20:52 |
| paul_ | well | 20:52 |
| paul_ | if we RC now | 20:53 |
| paul_ | historically that would mean we'd release in ~40 days ;p | 20:53 |
| paul_ | (or do an rc2) | 20:53 |
| paul_ | 1.0 has 5 rc's | 20:53 |
| paul_ | 1.1 had 3 | 20:53 |
| dhx_m | lol | 20:53 |
| paul_ | each rc takes about 40 days | 20:53 |
| paul_ | once we hit a release | 20:54 |
| paul_ | i.e a .X | 20:54 |
| paul_ | we seem to do a point release every 60 days | 20:54 |
| paul_ | i.e. | 20:54 |
| paul_ | 1st August would be 1.2 final | 20:55 |
| paul_ | 30th september would be 1.2.1 | 20:55 |
| paul_ | now in terms of 1.3 | 20:55 |
| dhx_m | 6 months for major releases? | 20:56 |
| dhx_m | and what about monthly updates? | 20:56 |
| dhx_m | or whenever they're needed (in terms of security fixes) | 20:56 |
| paul_ | 1.3a1 | 20:56 |
| dhx_m | I see no point delaying them | 20:56 |
| paul_ | we normally see 218 days after the first 1.2 release | 20:56 |
| paul_ | although we've speed up recently between 1.1.0 and 1.2a1 it was only 121 days | 20:57 |
| paul_ | so any time betweeen 20th October and Feb 2010 should be 1.3a1 | 20:58 |
| dhx_m | do we really need alpha releases? | 20:58 |
| paul_ | sure | 20:59 |
| paul_ | if we release an alpha on 20th October 2009 | 20:59 |
| dhx_m | IMO it'd be better to get early adopters to use git checkouts | 20:59 |
| dhx_m | *snapshots | 20:59 |
| paul_ | after the alpha on 20th October 2009 | 20:59 |
| dhx_m | it makes it easier to say "fixed, please update" | 20:59 |
| paul_ | we'd be for a final 1.3.0 during christmas vacation in January 2011 | 21:00 |
| paul_ | it could be quite nice to do a december 2010 1.3 release of mantis :) | 21:00 |
| paul_ | with snow and stuff | 21:00 |
| dhx_m | haha | 21:00 |
| dhx_m | well graphics are an area I want to improve in 1.3 | 21:01 |
| dhx_m | was thinking of Tango icons but they don't provide everything we need | 21:01 |
| paul_ | for the Xmas 2010 release, we'd need to support php 5.2+ if we allow 3 years of php versions | 21:02 |
| dhx_m | yep | 21:02 |
| paul_ | so we're set :) | 21:02 |
| paul_ | so now we've got a release schedule for 1.3 | 21:02 |
| dhx_m | what are the aims? | 21:02 |
| paul_ | need to decide contents | 21:03 |
| dhx_m | I recon a better automated test setup would be great | 21:03 |
| paul_ | so lets see... | 21:03 |
| paul_ | 1.3 alpha: 20th October 2009 | 21:03 |
| dhx_m | it can't take that long to write tests :) | 21:03 |
| paul_ | 1.3.0: Dec/Jan 2010/11 | 21:03 |
| paul_ | min php version 5.2 | 21:03 |
| dhx_m | better database and Unicode support | 21:03 |
| paul_ | contents | 21:03 |
| paul_ | a) tests | 21:03 |
| dhx_m | new graphics | 21:03 |
| paul_ | b) graphics | 21:03 |
| paul_ | c) auth plugins | 21:04 |
| dhx_m | :) | 21:04 |
| dhx_m | notification plugins? | 21:04 |
| dhx_m | or is that pushing it hah | 21:04 |
| dhx_m | actually documentation needs a lot of work too | 21:05 |
| paul_ | anyway | 21:06 |
| paul_ | I think the point is | 21:06 |
| paul_ | we need to decrease timescale | 21:06 |
| dhx_m | yep | 21:06 |
| paul_ | dhx_m: in terms of db_query and stuff you was asking about earlier | 21:07 |
| paul_ | I need to work out about binary files | 21:08 |
| paul_ | I think uploaded files wouldn't go through db_query_bound on one db | 21:08 |
| dhx_m | paul_: ok | 21:08 |
| dhx_m | yeah I'm a little worried about PHP script execution time too | 21:08 |
| paul_ | hmm? | 21:09 |
| dhx_m | if someone is on a slow connection uploading a file | 21:09 |
| sshirley | Hi all. In v1.1.0 can anyone tell me off the tops of their heads in which configuration file the "editing" threshold is maintained. By editing threshold, I mean under "resolved" issues, I don't have edit icon. I need to be able to open resolved issues. | 21:19 |
| nuclear_eclipse | search for read_only | 21:20 |
| paul_ | nuclear_eclipse: you missed the next release planning :) | 21:25 |
| nuclear_eclipse | no, I saw it | 21:25 |
| sshirley | Unfortunately that didn't help. I couldn't find that. I know I've asked you this before. But right now I just copied files over to the production server and that icon is now missing. Everything works fine on my development. I copied over 4 files (core/bug_api.php, core/constant_inc.php, config_defaults_inc.php, and custom_strings_inc.php). I have until 6pm. :-) | 21:27 |
| paul_ | nuclear_eclipse: it's statistically accurate :P | 21:28 |
| nuclear_eclipse | sshirley: $g_bug_readonly_status_threshold = RESOLVED; | 21:33 |
| sshirley | Thanks. JUST found it in config_defaults_inc.php. | 21:34 |
| nuclear_eclipse | sshirley: make sure you set the new value in config_inc.php so it doesn't get overwritten during an update | 21:35 |
| sshirley | So if it's in config_inc.php, then it overrides what's in config_defaults_inc.php? | 21:38 |
| sshirley | If I want everything to be "editable", I set it to the highest value? Meaning if 'Closed' in 90 and I set this variable, then 10,20,30,...,90, etc will be editiable? | 21:39 |
| nuclear_eclipse | yes, config_inc overrides config_defaults; that's its entire purpose :P | 21:39 |
| sshirley | Makes sense :-) | 21:39 |
| nuclear_eclipse | and no, whatever is *at* the threshold will not be editable; anything below it will be | 21:40 |
| paul_ | bug 8526 | 21:41 |
| mantisbot | Bug 8526 - vboctor - open - assigned | 21:41 |
| mantisbot | version_api should return date_order in unixtimestamp format - http://www.mantisbt.org/bugs/view.php?id=8526 | 21:41 |
| sshirley | Again in v1.1.0, where is the coloration in 'my views' set? Since I made my modifications, the green under Resolved is not showing (it's showing white) | 22:07 |
| sshirley | I think I found it :-) It's an array in config_defaults_inc.php, right? | 22:08 |
Generated by irclog2html.py