| ../irclogs/#mantishelp.2009-04-27.log | ||
| --- scribe started --- | 00:00 | |
| [KK]Kirill | hi | 07:10 |
|---|---|---|
| mantisbot | New bug: Bug 10410 - paontis - open - new | 09:09 |
| mantisbot | New bug: Upload and edit MS Office documents, they are corrupted - http://www.mantisbt.org/bugs/view.php?id=10410 | 09:09 |
| [KK]Kirill | giallu: you here? | 09:26 |
| giallu | ya | 09:26 |
| [KK]Kirill | hi.. How I can get priority value of given issue? | 09:27 |
| giallu | using soap? | 09:27 |
| giallu | or what? | 09:27 |
| [KK]Kirill | not. I want override function | 09:28 |
| [KK]Kirill | with write of priority status | 09:28 |
| mantisbot | New bug: Bug 10411 - paontis - open - new | 09:40 |
| mantisbot | New bug: Changes to project_view_state and view_state to create only private projects - http://www.mantisbt.org/bugs/view.php?id=10411 | 09:40 |
| [KK]Kirill | giallu: I try override function custom_function_override_roadmap_print_issue( $p_issue_id, $p_issue_level = 0 ) | 09:51 |
| [KK]Kirill | but I don't see any results. | 09:52 |
| giallu | ah ok, now I get it | 09:52 |
| giallu | well, to get the priority you use the bug_get_field function | 09:53 |
| mantisbot | New bug: Bug 10412 - karthisudar - open - new | 09:55 |
| mantisbot | New bug: defect - http://www.mantisbt.org/bugs/view.php?id=10412 | 09:55 |
| mantisbot | New bug: Bug 10413 - karthisudar - open - new | 09:55 |
| mantisbot | New bug: testing for LR - http://www.mantisbt.org/bugs/view.php?id=10413 | 09:55 |
| mantisbot | New bug: Bug 10414 - karthisudar - open - new | 09:55 |
| mantisbot | New bug: defect - http://www.mantisbt.org/bugs/view.php?id=10414 | 09:55 |
| giallu | grrrr | 09:55 |
| mantisbot | New bug: Bug 10415 - karthisudar - open - new | 10:00 |
| mantisbot | New bug: testing defects - http://www.mantisbt.org/bugs/view.php?id=10415 | 10:00 |
| mantisbot | New bug: Bug 10416 - karthisudar - open - new | 10:00 |
| mantisbot | New bug: testing defects - http://www.mantisbt.org/bugs/view.php?id=10416 | 10:00 |
| mantisbot | New bug: Bug 10417 - karthisudar - open - new | 10:15 |
| mantisbot | New bug: testing defects - http://www.mantisbt.org/bugs/view.php?id=10417 | 10:15 |
| mantisbot | New bug: Bug 10418 - karthisudar - open - new | 10:45 |
| mantisbot | New bug: testing defects - http://www.mantisbt.org/bugs/view.php?id=10418 | 10:45 |
| [KK]Kirill | ok. I did this | 10:47 |
| [KK]Kirill | giallu: thanks | 10:47 |
| mantisbot | New bug: Bug 10419 - jonathh - open - new | 11:05 |
| mantisbot | New bug: Versions are not sorted - http://www.mantisbt.org/bugs/view.php?id=10419 | 11:05 |
| thraxisp | giallu: can't we just disable this guy's account (karthisudar) | 11:07 |
| giallu | thraxisp, yeah | 11:09 |
| giallu | I think I'm going to hack the server and gain adminsitrator status... | 11:10 |
| [KK]Kirill | Who know git well? | 11:41 |
| [KK]Kirill | John, wake up | 11:55 |
| [KK]Kirill | giallu: close this bug 10384 | 12:08 |
| mantisbot | Bug 10384 - ruslan - open - new | 12:08 |
| mantisbot | Roadmap can/should show priority symbol for issues. - http://www.mantisbt.org/bugs/view.php?id=10384 | 12:08 |
| mantisbot | New bug: Bug 10420 - kesterhabermann - open - new | 12:10 |
| mantisbot | New bug: Custom fields with newline break CSV export - http://www.mantisbt.org/bugs/view.php?id=10420 | 12:10 |
| mantisbot | New bug: Bug 10421 - kesterhabermann - open - new | 12:15 |
| mantisbot | New bug: Default filename corrupted if project name contains special chars - http://www.mantisbt.org/bugs/view.php?id=10421 | 12:15 |
| nuclear_eclipse | morning all | 12:28 |
| [KK]Kirill | Hi | 12:28 |
| [KK]Kirill | John, nice to see you | 12:29 |
| nuclear_eclipse | btw, the green labels in gitweb mark local branches, pink marks remote branches, and yellow marks tags | 12:34 |
| [KK]Kirill | :) thanks | 12:40 |
| [KK]Kirill | nuclear_eclipse: close this bug 10384 | 12:41 |
| mantisbot | Bug 10384 - ruslan - open - new | 12:41 |
| mantisbot | Roadmap can/should show priority symbol for issues. - http://www.mantisbt.org/bugs/view.php?id=10384 | 12:41 |
| nuclear_eclipse | [KK]Kirill: I'm assuming you're requesting that 10384 be committed to master? | 13:07 |
| [KK]Kirill | nuclear_eclipse: Yes. How right request apply patch to master? | 13:07 |
| nuclear_eclipse | the "proper" method is to ask here or on the dev list; I just need to review it and/or make sure other devs agree that it should be included | 13:08 |
| [KK]Kirill | may be add resource where will be approved or disapproved patches? | 13:11 |
| nuclear_eclipse | would be a nice idea, but it's more of a concern of how to put something like that together without it just being a list of patch files... | 13:11 |
| [KK]Kirill | to view what thank each of all devs? | 13:12 |
| [KK]Kirill | ok.. I try to see what I can found of this topic | 13:12 |
| [KK]Kirill | John, I test your brunch strings | 13:13 |
| [KK]Kirill | It's work. | 13:13 |
| nuclear_eclipse | this is giving me a good idea for a new plugin though :) | 13:13 |
| [KK]Kirill | nuclear_eclipse: :) | 13:14 |
| nuclear_eclipse | [KK]Kirill: well, it "works" because I haven't actually hooked anything up to the new API functions yet :P | 13:14 |
| [KK]Kirill | nuclear_eclipse: I in my local brunch change mbfunction to string_* | 13:15 |
| nuclear_eclipse | I'm going to try some `sed` magic to see if I can't automate much of the conversion process.... | 13:16 |
| [KK]Kirill | :) | 13:16 |
| [KK]Kirill | I talk yesterday with ruslan | 13:16 |
| nuclear_eclipse | I noticed | 13:17 |
| [KK]Kirill | He has one wish | 13:18 |
| [KK]Kirill | for change version system | 13:18 |
| [KK]Kirill | Version [ 3] [b1] | 13:19 |
| [KK]Kirill | Version [ 3] [b2] | 13:19 |
| [KK]Kirill | Version [ 3] [b35] | 13:19 |
| [KK]Kirill | Version [ 3] [] << release (contain all change in beta version) | 13:19 |
| [KK]Kirill | and after release in changelog - delete all betas | 13:20 |
| nuclear_eclipse | well, that's more of a management issue - it's always possible for an admin to update the fixed-in version after the fact | 13:21 |
| [KK]Kirill | I don't think about it. | 13:22 |
| [KK]Kirill | How you set short numbers of mantis issue in mantisbt on your site? | 13:22 |
| nuclear_eclipse | or alternatively, a team could target issues for alpha/beta releases, but always set the fixed-in version to the final release | 13:22 |
| nuclear_eclipse | for our own site, we prefer to track what changed in each pre-release, and keep that history | 13:22 |
| [KK]Kirill | Ok | 13:25 |
| [KK]Kirill | How you set short numbers of mantis issue in mantisbt on your site? | 13:26 |
| nuclear_eclipse | what do you mean? | 13:26 |
| [KK]Kirill | In your site Issue number length 3 chars | 13:27 |
| [KK]Kirill | In main mantis 7 | 13:27 |
| [KK]Kirill | How you did this | 13:27 |
| [KK]Kirill | for example http://leetcode.net/mantis/view.php?id=31 | 13:29 |
| nuclear_eclipse | $g_display_bug_padding = 3; | 13:29 |
| nuclear_eclipse | $g_display_bugnote_padding = 3; | 13:29 |
| [KK]Kirill | sorry, I found. Thanks | 13:30 |
| [KK]Kirill | John, I right rollback last commit? see next | 13:36 |
| [KK]Kirill | git tag wplast | 13:36 |
| [KK]Kirill | //edit files | 13:37 |
| [KK]Kirill | git commit -a | 13:37 |
| [KK]Kirill | git formate-patch ... wplast > .... | 13:37 |
| [KK]Kirill | git reset --hard wplast | 13:38 |
| [KK]Kirill | ? | 13:38 |
| nuclear_eclipse | [KK]Kirill: it might be better to use branches for something like that, such as: | 13:39 |
| nuclear_eclipse | git checkout -b branchname | 13:39 |
| nuclear_eclipse | git commit | 13:39 |
| [KK]Kirill | I think you right | 13:39 |
| nuclear_eclipse | git format-patch master > .... | 13:39 |
| nuclear_eclipse | at point, you can checkout master again to be back at the "official" place, but still keep the branch with your local changes | 13:40 |
| paulr | . | 13:41 |
| [KK]Kirill | paulr: Hi | 13:42 |
| paulr | lo | 13:42 |
| paulr | my interent broke last night | 13:42 |
| * paulr couldn't push changes :( | 13:42 | |
| [KK]Kirill | why? | 13:42 |
| paulr | dunno :) | 13:42 |
| paulr | isp broke :) | 13:42 |
| nuclear_eclipse | hi paulr | 13:42 |
| [KK]Kirill | What now? | 13:42 |
| * paulr @ work | 13:46 | |
| [KK]Kirill | this task can be closed http://www.mantisbt.org/bugs/view.php?id=8268 | 13:46 |
| paulr | nuclear_eclipse: can i break trunk a bit? | 13:46 |
| paulr | (plsssssssssssssssssssss) | 13:46 |
| * [KK]Kirill *ROFL* | 13:46 | |
| * [KK]Kirill has 2 adsl internet @home :) | 13:47 | |
| nuclear_eclipse | paulr: how broken? | 13:48 |
| mantisbot | New bug: Bug 10422 - ruslan - open - new | 13:56 |
| mantisbot | New bug: [NEW] ability to mark some versions as ARCHIVE - http://www.mantisbt.org/bugs/view.php?id=10422 | 13:56 |
| giallu | nuclear_eclipse, ping | 14:12 |
| nuclear_eclipse | hi giallu | 14:12 |
| giallu | hi | 14:12 |
| giallu | can I kill the "this is a test" guy? | 14:12 |
| Kirill_Krasnov | giallu: You MUST KILL HIM :) | 14:13 |
| giallu | are you able to check if he made _any_ real bug report? | 14:13 |
| nuclear_eclipse | giallu: view issues page? | 14:14 |
| Kirill_Krasnov | nuclear_eclipse: too much test issues | 14:15 |
| Kirill_Krasnov | BAN HIM | 14:15 |
| nuclear_eclipse | according to view issues, he hasn't reported anything that hasn't yet been deleted :P | 14:15 |
| nuclear_eclipse | Kirill_Krasnov: I would if I had the access permissions to | 14:15 |
| giallu | nuclear_eclipse, I just hasck my account to admin level... | 14:16 |
| giallu | hacked even... | 14:16 |
| giallu | that's why I'm asking what to do | 14:16 |
| nuclear_eclipse | I was tempted to do the same | 14:16 |
| nuclear_eclipse | I suggest just disabling his account without actually deleting it | 14:16 |
| Kirill_Krasnov | nuclear_eclipse: may be add checked system for ip? | 14:16 |
| Kirill_Krasnov | ban for some submask | 14:17 |
| nuclear_eclipse | banning by IP is generally a very inefficient/poor method to handle that | 14:17 |
| giallu | ok. disabling it is (tbh, I don't see the point in keeping the account disabled. it just means we will not be able to register again with the same name) | 14:17 |
| nuclear_eclipse | disabling his account should be plenty | 14:17 |
| nuclear_eclipse | giallu: that's the point ;) | 14:17 |
| nuclear_eclipse | by disabling it, we at least prevent him recreating the account... | 14:18 |
| giallu | ok then... | 14:18 |
| nuclear_eclipse | I'm all for sending him a scathing email as well... | 14:18 |
| * giallu thinks about a feature to show a custom, per user, message when the account is disabled | 14:18 | |
| nuclear_eclipse | yes, that would be handy | 14:19 |
| nuclear_eclipse | eg, 'reason for being disabled' | 14:19 |
| Kirill_Krasnov | who has right in IRC? | 14:19 |
| nuclear_eclipse | I do | 14:19 |
| nuclear_eclipse | and paulr does | 14:19 |
| Kirill_Krasnov | kick [KK]Kirill | 14:19 |
| Kirill_Krasnov | :) | 14:21 |
| Kirill_Krasnov | Nickname is already in use.: [KK]Kirill | 14:21 |
| Kirill_Krasnov | :( | 14:21 |
| nuclear_eclipse | Kirill_Krasnov: in the future, if you register your nick with freenode, you can "recover" your account | 14:21 |
| Kirill_Krasnov | nuclear_eclipse: How? | 14:22 |
| nuclear_eclipse | /msg nickserv help | 14:22 |
| nuclear_eclipse | for now though, you'll either need to wait for your shadow to drop, or visit #help (iirc) and talk to a server op | 14:23 |
| Kirill_Krasnov | [17:25]Notice from NickServ (FreeNode): You may not ghost [KK]Kirill. | 14:23 |
| [KK]Kirill | nuclear_eclipse: Thanx | 14:27 |
| nuclear_eclipse | you're welcome | 14:27 |
| [KK]Kirill | :) | 14:27 |
| [KK]Kirill | nuclear_eclipse: If I create patch, I must create issue in mantsi before or just write in mail why I wish apply patch in dev build? | 14:44 |
| nuclear_eclipse | attaching it to an issue is probably the best method, but not required | 14:44 |
| [KK]Kirill | ok | 14:45 |
| mantisbot | New bug: Bug 10423 - Kirill - open - new | 15:07 |
| mantisbot | New bug: Get all issue from given user - http://www.mantisbt.org/bugs/view.php?id=10423 | 15:07 |
| giallu | [KK]Kirill, I beliece that is covered by an older "monitor user acitivity" bug report | 15:12 |
| [KK]Kirill | giallu: What issue number? I don't found issue | 15:12 |
| [KK]Kirill | giallu: I create patch for this | 15:13 |
| [KK]Kirill | I have local trouble | 15:13 |
| [KK]Kirill | #1146: Table 'kkrasnov.mantis_bug_revision_table' doesn't exist, запрос: SELECT COUNT(id) FROM mantis_bug_revision_table | 15:13 |
| [KK]Kirill | WHERE bug_id=3 AND type=3 AND bugnote_id=0. | 15:13 |
| [KK]Kirill | I create new mantis some mounth ago from git | 15:14 |
| [KK]Kirill | And table not created? | 15:14 |
| [KK]Kirill | please check new installation | 15:14 |
| giallu | bug 8157 | 15:15 |
| mantisbot | Bug 8157 - giallu - open - assigned | 15:15 |
| mantisbot | User activity report - http://www.mantisbt.org/bugs/view.php?id=8157 | 15:15 |
| [KK]Kirill | I forgot about note | 15:16 |
| [KK]Kirill | but I don't think that more needly | 15:17 |
| [KK]Kirill | upgrade database from admin - don't help - table not created | 15:21 |
| [KK]Kirill | where I can see structure of this table? | 15:25 |
| [KK]Kirill | I found in schema.php | 15:26 |
| [KK]Kirill | How I can execute create this table? | 15:27 |
| * [KK]Kirill go home | 15:30 | |
| paul_ | . | 17:20 |
| nuclear_eclipse | hi paul_ | 17:21 |
| paul_ | lo | 17:21 |
| paul_ | internet at home works :)) | 17:21 |
| nuclear_eclipse | you should register paulr as your nick, and boot out anyone who connects with it... | 17:21 |
| paul_ | I have | 17:21 |
| paul_ | nuclear_eclipse: I fixed some utf8 things over weekend | 18:28 |
| paul_ | how can i link the commits to bugs | 18:28 |
| nuclear_eclipse | "fix #xxx" | 18:28 |
| nuclear_eclipse | or "issue #xxx" | 18:28 |
| paul_ | after commit | 18:28 |
| nuclear_eclipse | have you already pushed the commits to official repo? | 18:29 |
| paul_ | nope | 18:29 |
| nuclear_eclipse | then you can a) use `rebase -i` and/or `commit --amend` to edit the commit messages, or b) push to repo, and manually link the commits to the issues through the source integration interface | 18:30 |
| paul_ | b sounds easier | 18:30 |
| nuclear_eclipse | a is rather easy to | 18:30 |
| paul_ | can we talk about helper_ensure_post briefly | 18:31 |
| nuclear_eclipse | maybe | 18:31 |
| paul_ | # helper_ensure_post(); | 18:31 |
| paul_ | in query_delete.php | 18:31 |
| paul_ | <form method="post" action="<?php print $t_delete_url; ?>"> | 18:32 |
| paul_ | <br /><br /> | 18:32 |
| paul_ | <input type="hidden" name="source_query_id" value="<?php print $f_query_id; ?>"/> | 18:32 |
| paul_ | <input type="submit" class="button" value="<?php print lang_get( 'delete_query' ); ?>"/> | 18:32 |
| paul_ | </form> | 18:32 |
| paul_ | there's no form security stuff there right? | 18:32 |
| nuclear_eclipse | right | 18:32 |
| paul_ | is it just the case of doing: | 18:32 |
| paul_ | echo form_security_field("somefoo") | 18:33 |
| paul_ | in the <form> tags | 18:33 |
| nuclear_eclipse | that's half of it | 18:33 |
| paul_ | and then just calling form_security_validate("somefoo") instead of helper_ensure_post | 18:33 |
| nuclear_eclipse | and form_security_purge("somefoo") after all the processing has been completede | 18:33 |
| paul_ | any thing else to be aware of ? | 18:34 |
| nuclear_eclipse | just make sure that "somefoo" is unique among all the other form security strings for that specific action | 18:35 |
| paul_ | and now last thing | 18:35 |
| paul_ | last night, I went back to looking at my bugobject branch | 18:35 |
| nuclear_eclipse | in this case, "query_delete" maintains the current habit of naming the security string after the processing page | 18:35 |
| paul_ | and had to do a merge | 18:35 |
| paul_ | which conflicted | 18:35 |
| paul_ | can i collapse the commits in this branch into one? | 18:37 |
| nuclear_eclipse | yeah, either using `rebase -i` and setting all commits after the first to 'squash', or by using `merge --squash` | 18:39 |
| paul_ | so if I wanted to merge/squash bugobjects branch into bugobjects2 branch | 18:39 |
| paul_ | how would I do this :) | 18:40 |
| nuclear_eclipse | `git co -b bugobjects2 master; git merge --squash bugobjects` | 18:40 |
| paul_ | $ git merge --squash bugobjects | 18:41 |
| paul_ | Auto-merging view_all_bug_page.php | 18:41 |
| paul_ | CONFLICT (content): Merge conflict in view_all_bug_page.php | 18:41 |
| paul_ | Squash commit -- not updating HEAD | 18:41 |
| paul_ | Automatic merge failed; fix conflicts and then commit the result. | 18:41 |
| nuclear_eclipse | congrats | 18:41 |
| paul_ | ... | 18:41 |
| paul_ | stupid thing | 18:43 |
| paul_ | nice | 18:44 |
| paul_ | nuclear_eclipse: sorted :) | 18:45 |
| Seablade | Hey quick question, we are getting a couple of users reporting being unable to login or change passwords currently in an installation of Mantis, they are getting "Message: APPLICATION WARNING #2702: Your session has become invalidated." What would be causing this, and is there a way to address it(Fix or workaround?) I didn't find much in the BT. | 18:46 |
| paul_ | Seablade: speak to nuclear_eclipse | 18:46 |
| nuclear_eclipse | Seablade: are you using a proxy server? | 18:47 |
| Seablade | nuclear_eclipse: For the hosting or the client? | 18:47 |
| nuclear_eclipse | either | 18:47 |
| paulr | the client user | 18:47 |
| nuclear_eclipse | Seablade: also, what verson of Mantis? | 18:47 |
| Seablade | nuclear_eclipse: I can't speak for the client, but can check. The hosting is on a shared host, but I don't think there is a reverse proxy in effect there, I could be wrong | 18:47 |
| Seablade | nuclear_eclipse: 1.2.0a3 | 18:47 |
| Seablade | nuclear_eclipse: Dreamhost | 18:48 |
| nuclear_eclipse | Seablade: you can disable the session validation from the configuration file, but note that it opens a security hole, allowing potential session hijacking | 18:48 |
| Seablade | nuclear_eclipse: Is this a common issue if a proxy is involved? | 18:49 |
| * paulr wants to know what proxy is in the loop btw | 18:49 | |
| nuclear_eclipse | Seablade: it's a problem for any proxy server that doesn't obey the caching headers that Mantis sends | 18:49 |
| nuclear_eclipse | or wait | 18:49 |
| nuclear_eclipse | that's a different issue | 18:50 |
| nuclear_eclipse | the session issue happens with any client that connects to Mantis from a load-balanaced proxy server, where each connection can look like it's coming from a different IP address | 18:50 |
| * Seablade is checking on the presense of a proxy with one of the users that reported the problem now, assuming he got the right user | 18:50 | |
| paulr | $t_columns = get_class_vars ('BugData'); | 18:52 |
| paulr | returns the default values too??? | 18:52 |
| nuclear_eclipse | Return Values | 18:52 |
| nuclear_eclipse | Returns an associative array of default public properties of the class. The resulting array elements are in the form of varname => value. | 18:52 |
| Seablade | Hmm I don't think Dreamhost is set up like that, in as far as they do have individual servers that they are sharing, not really a cluster setup in the way MediaTemple offers to my knowledge. However the user I just checked with is having issues going through an iPhone getting that error, so it is a good bet that they are indeed running through a proxy | 18:52 |
| nuclear_eclipse | php.net/manual ftw... | 18:52 |
| nuclear_eclipse | Seablade: the problem is most likely caused by a) EDGE/3G disconnecting/reconnecting with a new IP address, and/or b) hopping between wifi and EDGE/3G and back, etc | 18:54 |
| nuclear_eclipse | it's an eventual goal to implement the user option to disable IP-based session validation at login time | 18:54 |
| paulr | nuclear_eclipse: that just leaves bug api | 18:55 |
| nuclear_eclipse | or implement some other basis for session validation to help prevent hijacking - see bug 9744 on the official tracker | 18:55 |
| mantisbot | Bug 9744 - seiji - open - assigned | 18:55 |
| mantisbot | Users behind proxy will get 'ERROR_SESSION_NOT_VALID' - http://www.mantisbt.org/bugs/view.php?id=9744 | 18:55 |
| paulr | see john this is the problem with never doing a release :) | 18:56 |
| paulr | users run beta's :) | 18:56 |
| Seablade | nuclear_eclipse: Ok, thanks. I will check with the other user[s] that report this issue and see if we can get it sorted out without disabling the session validation preferably. Ill probably end up being back in the future with other possible problems:) Thanks for the help | 18:56 |
| paulr | feel free to help out with suggestsions of fixes btw :) | 18:57 |
| Seablade | paulr: Heh not sure why this project switched to the alpha, I am sure there was reasoning, but yea we upgraded since then based off the issues with web browsers, etc. | 18:57 |
| Seablade | paulr: Heh anytime I can I certainly will, it is just a matter of whether I can or not, not the webmaster for this, just someone that helps out with the bugs:) | 18:57 |
| nuclear_eclipse | Seablade: you're welcome | 18:58 |
| Wildman | afternoon everyone | 18:58 |
| Wildman | Giallu are you listening? | 18:58 |
| paulr | hi | 18:59 |
| paulr | - $t_status = config_get( 'bug_submit_status' ); | 19:06 |
| paulr | + $t_starting_status = config_get( 'bug_submit_status' ); | 19:06 |
| paulr | + $t_original_status = $c_status; | 19:06 |
| paulr | nuclear_eclipse: what's this crap | 19:06 |
| nuclear_eclipse | context? | 19:07 |
| paulr | some commit | 19:07 |
| nuclear_eclipse | that helps | 19:08 |
| Wildman | giallu this is Bill, we talked Friday regarding the import issues plugin if you have time to speak with me I have a couple of questions | 19:09 |
| paulr | nuclear_eclipse: what time are you leaving tonight? | 19:10 |
| nuclear_eclipse | paulr: in about 10 minutes, for class :P | 19:10 |
| nuclear_eclipse | Wildman: giallu is in Italian timezone, so he may not be around until a bit later, or much later :P | 19:11 |
| paulr | nuclear_eclipse: are you/will you beback after? | 19:12 |
| nuclear_eclipse | perhaps | 19:13 |
| nuclear_eclipse | you can always just ping me in a couple hours and see if I respond :P | 19:14 |
| Wildman | thanks Nuclear | 19:14 |
| Wildman | I talked to him last Friday around this time I think he was trying to get his kids to sleep LOL | 19:14 |
| paulr | I might just do bigpush tomorrow :) | 19:14 |
| nuclear_eclipse | paulr: can you double check bug 10369? | 19:16 |
| mantisbot | Bug 10369 - Kirill - open - assigned | 19:16 |
| mantisbot | Verification link not work - http://www.mantisbt.org/bugs/view.php?id=10369 | 19:16 |
| paulr | i think your right | 19:16 |
| paulr | or well | 19:16 |
| paulr | I was doing a "it's html it should be & | 19:16 |
| paulr | but you mgiht be right: ) | 19:16 |
| nuclear_eclipse | I think we need to revert your "fix" commit, as you were fixing a broken mail client and breaking everything else :P | 19:16 |
| paulr | I was wondering if we could just come up with a sequence and split it | 19:16 |
| paulr | e.g. | 19:16 |
| paulr | atm, we do something like | 19:17 |
| paulr | username=foo&key=dfjklsdjkl or something right? | 19:17 |
| paulr | so if we did say | 19:17 |
| paulr | key=foo___***___dfjklsdjkl | 19:17 |
| paulr | or something | 19:17 |
| paulr | we could fix both | 19:17 |
| paulr | i.e. take some char that isn't allowed in a username | 19:17 |
| paulr | and use it as a split | 19:18 |
| nuclear_eclipse | hmm, that sounds like a dirty hack; I'd much rather not... | 19:18 |
| paulr | well, stupid question | 19:18 |
| paulr | surely it should be & though? | 19:18 |
| nuclear_eclipse | we can just tell Entourage users to complain about getting the bug fixed in Entourage, rather than pandering to them... | 19:18 |
| nuclear_eclipse | no, the emails aren't HTML, so & is invalid | 19:18 |
| nuclear_eclipse | the only reason you need & in HTML is because HTML is annoying... | 19:19 |
| paulr | http://www.tech-archive.net/Archive/Mac/microsoft.public.mac.office.entourage/2004-04/0558.html | 19:19 |
| nuclear_eclipse | anywho, I gotta run | 19:20 |
| nuclear_eclipse | cheers | 19:20 |
| mantisbot | New bug: Bug 10424 - cbradney - open - new | 19:43 |
| mantisbot | New bug: Requesting option to disable the use of some versions from the list - http://www.mantisbt.org/bugs/view.php?id=10424 | 19:43 |
| giallu | Wildman, I'm around now | 20:07 |
| giallu | too late... | 20:07 |
| paulr | lo giallu | 20:16 |
| giallu | hi paulr | 20:17 |
| giallu | what kind of brakeage are you planning? | 20:17 |
| paulr | the normal | 20:18 |
| paulr | i've edited a regex to do with url's for utf8 | 20:18 |
| paulr | and started merging one of my branches in properly | 20:18 |
| giallu | which one? | 20:22 |
| nuclear_eclipse | paulr: that sounds like it should be reviewud before pushing... | 20:22 |
| paulr | bugobjectone | 20:22 |
| giallu | nuclear_eclipse, since when we review _before_ pushing? | 20:22 |
| giallu | :P | 20:22 |
| paulr | nuclear_eclipse: it's already had some input :) | 20:22 |
| paulr | tbh, i might make a bundle tpye thing | 20:23 |
| nuclear_eclipse | utf8 just seems like one of those things that's liable to make things even worse... :P | 20:23 |
| nuclear_eclipse | paulr: please don't make bundles, just make formatted patches, they're *so* much easier to work with | 20:23 |
| paulr | your just lazy | 20:24 |
| paulr | I thought the point of bundles is they just apply to your repo | 20:27 |
| paulr | :) | 20:27 |
| nuclear_eclipse | the point of bundles is to let one repo transfer data to another, not for human use | 20:31 |
| nuclear_eclipse | formatted patches and remote references are what humans should use... | 20:31 |
| nuclear_eclipse | as the manpage for git-bundle says: "This command provides support for git-fetch and git-pull to operate..." | 20:32 |
| CIA-11 | Mantisbt: s.mazeland * re9cbf60f8c1b / (10 files in 5 dirs): Fix: #0007484: implement RTL css | 20:33 |
| nuclear_eclipse | ie, it's a plumbing command, not porcelain | 20:33 |
| paulr | do we really need a config variable for RTL css ;/ | 20:36 |
| giallu | no, please, not another one.... | 20:36 |
| nuclear_eclipse | apparently we do........ =\ | 20:37 |
| giallu | can we state a goal of, let's say, max 5 config variables for mantis 2.0 ? | 20:37 |
| paulr | lol | 20:37 |
| * nuclear_eclipse beats giallu with a big stick for mentioning 2.0 | 20:37 | |
| giallu | hey nuclear_eclipse, if you like IO can propose that for 1.2 | 20:38 |
| giallu | so we release in 2012 | 20:38 |
| * nuclear_eclipse hits head on desk... | 20:38 | |
| paulr | nuclear_eclipse: review | 20:38 |
| paulr | pls | 20:38 |
| paulr | k | 20:38 |
| paulr | thx | 20:38 |
| paulr | wheres daryn | 20:38 |
| giallu | daryn hitted desk too strongly | 20:39 |
| paulr | do we need to call string_attribute before we put it into a dropdown list? | 20:41 |
| nuclear_eclipse | paulr: I need a URL before I can review anything.... | 20:41 |
| nuclear_eclipse | paulr: no, string_line should suffice | 20:41 |
| paulr | no i mean for editing | 20:41 |
| paulr | i.e. | 20:42 |
| paulr | <?php print_category_option_list( $t_bug->category_id, $t_bug->project_id ) ?> | 20:42 |
| paulr | that would normally be: | 20:42 |
| paulr | print_category_option_list( string_attribute( $t_bug->category_id ), $t_bug->project_id ); | 20:42 |
| nuclear_eclipse | I don't think you need it in that case... | 20:43 |
| paulr | gimme10min | 20:43 |
| paulr | <?php print_version_option_list( $t_bug->version, $t_bug->project_id, $t_product_version_released_mask ); ?> | 20:46 |
| paulr | what about that case? | 20:46 |
| paulr | yes? | 20:46 |
| nuclear_eclipse | I don't know off hand, but I would think not | 20:53 |
| nuclear_eclipse | the print_*_list functions sholud apply any stirng_* functions as necessary | 20:54 |
| paulr | how do i push a branch to a fork again | 20:57 |
| paulr | (on mforge | 20:57 |
| siebrand | nuclear_eclipse: john, any planning for http://www.mantisbt.org/bugs/view.php?id=10271 (mb_string warning) | 21:01 |
| * paulr doesn't think we should require mb_string | 21:02 | |
| siebrand | paulr: for proper support for certain methods in many languages, it is almost unavoidable. We need to devise some way to warn people if they choose a language that needs it. | 21:04 |
| siebrand | paulr: only 15% of the world population understands English. | 21:04 |
| paulr | sure | 21:05 |
| paulr | at the same time, in some cases (over weekend) things I notice that we break on | 21:05 |
| paulr | dont require mb_string to fix | 21:05 |
| paulr | for example as 2 examples: | 21:06 |
| paulr | in our email api we were using wordwrap to wrap the email width to 75 characters | 21:06 |
| paulr | then using phpmailer's wordwrap functionality (which supports utf8 supposedly) to wrap to 80 characters | 21:06 |
| paulr | -- | 21:06 |
| paulr | 2nd example: the url parsing stuff nukes utf8 which again I think i've fixed over weekend | 21:07 |
| siebrand | yeah, it's full of bugs. Let's tackle them one at a time, and not forget that making MantisBT available in languages other than Lantin scripted is a Good Thing (tm) | 21:07 |
| giallu | paulr, your examples are fine, but sibrand is right. we can't support all languages without mb_* | 21:08 |
| paulr | giallu: depending on what we are doing | 21:08 |
| giallu | so we definitely need to prevent users from using mantis with non-english languages if mb_String is not there | 21:08 |
| giallu | note: | 21:08 |
| siebrand | btw, why is it that if I choose to sort on issue id in http://www.mantisbt.org/bugs/view_all_bug_page.php?filter=3255091 that I go from 28 results to 5? | 21:08 |
| giallu | not even that guarantees we have correct utf handling | 21:09 |
| paulr | as a 3rd example | 21:10 |
| paulr | strlen() | 21:10 |
| paulr | (which doesn't support utf8) | 21:10 |
| paulr | my understanding is there's 3 possible ways to do a utf8 strlen: | 21:10 |
| paulr | 1) call mb_strlen(&$foo, 'UTF-8'); | 21:11 |
| paulr | 2) call iconv_strlen(&$foo, 'UTF-8'); | 21:11 |
| paulr | 3) call strlen(utf8_decode(&$string)); | 21:11 |
| paulr | if we chose option 3, it would be slower, but would work for everyone even without mb_* | 21:13 |
| giallu | tbh, 2 and 3 sound like workarounds I don't really want to see | 21:13 |
| siebrand | strtoupper (which I have on 257 lines on our code) completely messes up everything to do with Cyrillic. Compare it to having random X-es in your UI. | 21:14 |
| giallu | siebrand, right. but there are more | 21:14 |
| * siebrand nods knowingly. | 21:14 | |
| paulr | giallu: iconv isn't... :) | 21:14 |
| giallu | paulr, so are we planning to outsmart php guys here? | 21:14 |
| siebrand | mb_string is a solution for a gamut of unicode issues. | 21:15 |
| siebrand | iconv is the work around. | 21:15 |
| siebrand | what exactly is the con for mb_string? | 21:15 |
| giallu | I have no problem with it... expecially because is for non-english locales | 21:16 |
| siebrand | IMO we just require it, and be done with it. It will enable to improve out product tremendously with regards to language support. Where will its quality decrease? | 21:16 |
| paulr | there's not that much that breaks without it atm | 21:18 |
| paulr | some of the things that break | 21:18 |
| paulr | are our regex's | 21:18 |
| * giallu sighs... | 21:18 | |
| paulr | mb_string does't help with that | 21:19 |
| siebrand | paulr, you are being Anglo-centric to the bone. | 21:19 |
| paulr | no | 21:19 |
| siebrand | why not? | 21:19 |
| paulr | i've spent a fair amount of time recently starting to look at the issues that are coming up | 21:19 |
| giallu | if nothing is broken, can you explain http://tinyurl.com/cegwwy ? | 21:21 |
| giallu | and this is the tip.. | 21:21 |
| giallu | btw, I'm tired... | 21:22 |
| paulr | giallu: can I exclude the closed issues when responding to this? :) | 21:23 |
| giallu | don't bother | 21:23 |
| paulr | one of the main problems | 21:24 |
| paulr | that I commented on over weekend | 21:24 |
| paulr | paul_http://www.mantisbt.org/bugs/view.php?id=926912:48 | 21:24 |
| paulr | paul_^^ that seems to be 'biggest' problem12:49 | 21:24 |
| paulr | paul_we should force utf8 no?12:49 | 21:24 |
| paulr | dhx_mI think so13:06 | 21:24 |
| paulr | dhx_mI can't see any reason why you wouldn't be using UTF8 | 21:24 |
| paulr | bug 9269 | 21:24 |
| mantisbot | Bug 9269 - 0xHH - open - acknowledged | 21:24 |
| mantisbot | On installation mantis create database with default collation settings. - http://www.mantisbt.org/bugs/view.php?id=9269 | 21:24 |
| paulr | i.e. atm | 21:25 |
| paulr | if you upgrade/create a db | 21:25 |
| paulr | it doesn't necessarily use utf8 as type | 21:25 |
| giallu | that's _another_ problem | 21:28 |
| giallu | that we should try to fix | 21:28 |
| paulr | nod | 21:28 |
| paulr | i've got some ideas on that | 21:28 |
| paulr | i'm just waiting confirmation on if we should *always* be telling mysql utf8? | 21:28 |
| giallu | yes we should | 21:29 |
| paulr | (for new installs) | 21:29 |
| siebrand | paulr: from what/whom? It's YES! | 21:29 |
| giallu | ascii is a subset of utf | 21:29 |
| giallu | so going utf you don't break anyone | 21:29 |
| giallu | but fix those non ascii | 21:29 |
| giallu | so it's a must | 21:30 |
| paulr | siebrand: one of my weekend work was to fix(or at least try to) the string_insert_href function | 21:30 |
| paulr | that parses urls | 21:30 |
| siebrand | giallu: <unknown> will have to export db, convert, import before being able to update. | 21:30 |
| paulr | and what not to work with utf8 | 21:30 |
| siebrand | paulr: would be resolved if we *require* a db convert to UTF 8 for 1.2 branch and up. | 21:30 |
| paulr | nod | 21:31 |
| giallu | paulr, I had a working conversion script | 21:31 |
| siebrand | paulr: and we could drop all the non-utf locales that are there for nothing anyway at the moment... | 21:31 |
| giallu | but never had time to convert to something usable by mantis | 21:31 |
| siebrand | btw, any reason why I should not drop those in master anyway? | 21:31 |
| giallu | siebrand, if we have the corresponding utf one, I'd say kill'em | 21:32 |
| siebrand | giallu: we do. | 21:32 |
| siebrand | ok, easy enough :) | 21:32 |
| siebrand | giallu: poof. They're gone. | 21:59 |
| CIA-11 | Mantisbt: s.mazeland * re593fed9cecc /lang/ (32 files): Remove obsolete and outdated non-UTF8 language files. All have a better maintained UFT8 equivalent. | 22:00 |
| giallu | :D | 22:00 |
| siebrand | finally it's easy to count that we have 44 locales. | 22:01 |
| giallu | we should add it to the website... | 22:01 |
| nuclear_eclipse | just to add my two cents, I think we should check/warn for installations without mb_*, but still support them by using standard string functions for cases where the hosting environment doesn't have the multibyte PHP extensions, and the site is never planning to use anything other than English anyways; ie, we should be as compatilble as possible while still warning/recommending having the multibyte extension | 22:19 |
| nuclear_eclipse | which is exactly the way I'm headed with my work for bug 10303 | 22:20 |
| mantisbot | Bug 10303 - jreese - open - assigned | 22:20 |
| mantisbot | Create and use string_api wrappers for multibyte string functions - http://www.mantisbt.org/bugs/view.php?id=10303 | 22:20 |
| paulr | nuclear_eclipse: http://git.mantisforge.org/w/mantisbt/paul.git?a=commitdiff;h=83784b10182a64013c0cdeb6ed0f4ddf737fe68d | 22:23 |
| paulr | nuclear_eclipse: http://git.mantisforge.org/w/mantisbt/paul.git?a=commitdiff;h=5226fbc4b721dd47627c987e57d2c7d5c5a1431a | 22:24 |
| * nuclear_eclipse looks | 22:29 | |
| paulr | feel free to look at rest of that branch | 22:29 |
| nuclear_eclipse | the wordwrap diff looks fine to me | 22:32 |
| nuclear_eclipse | what's with the $t_url_hex? | 22:32 |
| paulr | me trying to work out what the regex does ;p | 22:33 |
| nuclear_eclipse | I'm just not sure why you need the hex stuff since it's already covered by url_valid_chars.... | 22:34 |
| paulr | you had it :) | 22:34 |
| paulr | that's a null change | 22:34 |
| nuclear_eclipse | ... I don't remember that... | 22:34 |
| paulr | all i've done is | 22:34 |
| paulr | a) added /pL | 22:35 |
| paulr | b) add /u | 22:35 |
| paulr | erm | 22:35 |
| paulr | make that \pL | 22:35 |
| paulr | so /se becomes /sue | 22:35 |
| paulr | and [:alnum:] is now followed with a \pL which apparently is unicode letters | 22:35 |
| nuclear_eclipse | what's \pL ? | 22:35 |
| paulr | apparently utf8 leters of any case | 22:36 |
| nuclear_eclipse | well, I gotta go eat dinner, I'll continue looking at it later | 22:37 |
| CIA-11 | Mantisbt: s.mazeland master-1.1.x * r857ebb166f78 / (config_defaults_inc.php lang/strings_arabic.txt): Fix #0010405: Added Arabic localisation. Contributed by kawzaki. | 22:42 |
| CIA-11 | Mantisbt: s.mazeland master-1.1.x * r9f5f8c95b4fa /lang/strings_polish.txt: Fix syntax error in strings_polish.txt. Where did that come from? | 22:45 |
Generated by irclog2html.py