| ../irclogs/#mantishelp.2009-04-22.log | ||
| --- scribe started --- | 00:00 | |
| truonglong88 | hi | 02:54 |
|---|---|---|
| truonglong88 | plz tell me the address of bug_page_view | 02:55 |
| dhx_m | hi | 02:57 |
| dhx_m | ? | 02:57 |
| mantisbot | New bug: Bug 10353 - ispolin - open - new | 03:44 |
| mantisbot | New bug: ??? - http://www.mantisbt.org/bugs/view.php?id=10353 | 03:44 |
| mantisbot | New bug: Bug 10354 - karthisudar - open - new | 07:00 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10354 | 07:00 |
| mantisbot | New bug: Bug 10355 - karthisudar - open - new | 07:05 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10355 | 07:05 |
| mantisbot | New bug: Bug 10356 - karthisudar - open - new | 07:05 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10356 | 07:05 |
| mantisbot | New bug: Bug 10357 - karthisudar - open - new | 07:05 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10357 | 07:05 |
| dhx_m | oh dear :) | 07:07 |
| giallu__ | grrrr | 07:18 |
| [KK]Kirill | John, can you update git://git.mantisforge.org/mantisbt/jreese.git to last trank | 07:31 |
| [KK]Kirill | ? | 07:32 |
| dhx_m | for the strings branch? | 07:40 |
| [KK]Kirill | yes | 07:41 |
| dhx_m | is there a merge conflict? | 07:42 |
| [KK]Kirill | not.. to much diff | 07:43 |
| dhx_m | I ask because I haven't merged stuff before | 07:44 |
| dhx_m | I would have thought that would have merged easily | 07:44 |
| dhx_m | given it is 2 patches that don't look like they'd be affected by the latest master head | 07:44 |
| [KK]Kirill | can you upload patch for last master? | 07:48 |
| dhx_m | I thought with git you normally don't have to merge the master into branches? | 07:49 |
| [KK]Kirill | good question | 07:49 |
| [KK]Kirill | I don't think yet | 07:49 |
| [KK]Kirill | one sec | 07:50 |
| mantisbot | New bug: Bug 10358 - karthisudar - open - new | 07:50 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10358 | 07:50 |
| dhx_m | is he trying to count to 200 or something? | 07:51 |
| [KK]Kirill | Ban him | 07:52 |
| dhx_m | also you probably want to be using "git rebase master" in your working branch | 07:52 |
| dhx_m | rather than the merge command | 07:52 |
| dhx_m | ... if you need to have the latest master commits in your branch | 07:53 |
| [KK]Kirill | I press git commit -a | 07:53 |
| [KK]Kirill | but not update | 07:53 |
| [KK]Kirill | discard changes? | 07:53 |
| dhx_m | for a lot of small fixes that aren't affected by changes in master since you started the branch, you shouldn't need to merge master into your branch, or rebase your branch | 07:53 |
| dhx_m | so you've merged the strings branch into master? | 07:54 |
| dhx_m | but haven't pushed yet? | 07:54 |
| [KK]Kirill | not | 07:54 |
| [KK]Kirill | I merge master to strings branch | 07:54 |
| dhx_m | ah | 07:54 |
| dhx_m | yep discard | 07:54 |
| [KK]Kirill | I try coomit strings branch | 07:54 |
| [KK]Kirill | ok | 07:54 |
| dhx_m | then checkout the master | 07:55 |
| dhx_m | and type 'git pull . strings' | 07:55 |
| dhx_m | that should merge the strings branch into master | 07:55 |
| dhx_m | it should tell you if the automatic merging didn't work | 07:56 |
| [KK]Kirill | one sec | 07:57 |
| [KK]Kirill | I commit master to strings :( | 07:57 |
| dhx_m | one of the main benefits of git is that merging branches is 100000000000000000 times easier than with SVN or similar | 07:57 |
| dhx_m | no problem (I think) | 07:57 |
| [KK]Kirill | how I can revert last commit? | 07:57 |
| dhx_m | "git log" to get the previous commit id (sha1 hash) | 07:58 |
| dhx_m | then you can "git checkout ID" where ID is the ID of the commit you want to go back to | 07:58 |
| dhx_m | wait maybe this isn't the best way | 07:58 |
| [KK]Kirill | It's normal ? | 07:59 |
| [KK]Kirill | $ git checkout e50944ce028886db539855a0d7d0acbf5dc3a94b | 07:59 |
| [KK]Kirill | Note: moving to "e50944ce028886db539855a0d7d0acbf5dc3a94b" which isn't a local branch | 07:59 |
| [KK]Kirill | If you want to create a new branch from this checkout, you may do so | 07:59 |
| [KK]Kirill | (now or later) by using -b with the checkout command again. Example: | 07:59 |
| [KK]Kirill | git checkout -b <new_branch_name> | 07:59 |
| [KK]Kirill | HEAD is now at e50944c... Perf: allow passing in charset to avoid calling lang/config_get in loops. Perf: Replace string_display_line with call to string_html_s | 07:59 |
| [KK]Kirill | pecialchars( string_strip_hrefs in print_project_option list to avoid calling plugin/config loops | 07:59 |
| dhx_m | ah | 07:59 |
| dhx_m | do this: | 07:59 |
| dhx_m | git reset --hard SHA1_HASH | 07:59 |
| [KK]Kirill | HEAD is now at e50944c Perf: allow passing in charset to avoid calling lang/config_get in loops. | 08:01 |
| dhx_m | it'll go back in time and make your working copy exactly the same as what it was like back at revision SHA1_HASH | 08:01 |
| dhx_m | ok | 08:01 |
| dhx_m | that was after the git reset command? | 08:01 |
| [KK]Kirill | HEAD is now at e50944c Perf: allow passing in charset to avoid calling lang/config_get in loops. | 08:02 |
| dhx_m | sounds OK to me | 08:02 |
| dhx_m | now you can git pull . strings | 08:03 |
| [KK]Kirill | in msaster branch ? | 08:04 |
| [KK]Kirill | master* | 08:04 |
| dhx_m | yep | 08:04 |
| dhx_m | assuming you have done a git reset | 08:04 |
| [KK]Kirill | can I copy this branch | 08:04 |
| dhx_m | the master? or strings? | 08:04 |
| [KK]Kirill | I don't want change master branch | 08:04 |
| [KK]Kirill | master | 08:04 |
| dhx_m | ah ok, just create a new branch from master | 08:04 |
| dhx_m | and merge strings into that branch (I think) | 08:05 |
| dhx_m | but your master should show "nothing to do" for "git status" | 08:05 |
| dhx_m | also btw don't use git reset if you have already pushed your commits to someone else | 08:06 |
| dhx_m | because then they won't be able to trace those commits back to your repo (in your repo, those commits are lost for good... no record) | 08:06 |
| dhx_m | http://www.kernel.org/pub/software/scm/git/docs/git-reset.html explains it better | 08:07 |
| [KK]Kirill | $ git pull .strings | 08:08 |
| [KK]Kirill | fatal: '.strings' does not appear to be a git repository | 08:08 |
| [KK]Kirill | fatal: The remote end hung up unexpectedly | 08:08 |
| dhx_m | nah, space needed after the period ;) | 08:08 |
| dhx_m | you're saying "pull the strings branch into my current working branch" | 08:08 |
| dhx_m | so git pull {destination} {source} | 08:09 |
| [KK]Kirill | ah.. sorry | 08:09 |
| [KK]Kirill | $ git pull . strings | 08:09 |
| [KK]Kirill | From . | 08:09 |
| [KK]Kirill | * branch strings -> FETCH_HEAD | 08:09 |
| [KK]Kirill | Already up-to-date. | 08:09 |
| dhx_m | did you "git checkout {branch you want to merge to}"? | 08:09 |
| [KK]Kirill | Yes. I in branch mastertest | 08:10 |
| [KK]Kirill | witch I create in master by command git branch mastertest | 08:10 |
| mantisbot | New bug: Bug 10359 - karthisudar - open - new | 08:10 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10359 | 08:10 |
| dhx_m | did you use "checkout" after the "branch" command | 08:11 |
| dhx_m | because "branch" does not automatically checkout the new branch | 08:11 |
| dhx_m | from the error you got, it sounds like you were in the strings branch | 08:12 |
| [KK]Kirill | Yes. I checkout to this branch | 08:14 |
| dhx_m | hmmm I'll try it myself to see what the problem could be | 08:14 |
| dhx_m | hmmm well I can merge OK with the procedure I outlined | 08:15 |
| [KK]Kirill | one sec | 08:15 |
| dhx_m | merging my 10330 branch into a new branch I created from the current master | 08:15 |
| dhx_m | I did: | 08:15 |
| dhx_m | git checkout master | 08:15 |
| dhx_m | git pull | 08:15 |
| dhx_m | git branch testmerge | 08:16 |
| dhx_m | git checkout testmerge | 08:16 |
| dhx_m | git pull . 10330 | 08:16 |
| dhx_m | and that worked OK for me | 08:16 |
| [KK]Kirill | $ git pull . strings | 08:19 |
| [KK]Kirill | From . | 08:19 |
| [KK]Kirill | * branch strings -> FETCH_HEAD | 08:19 |
| [KK]Kirill | Auto-merging core/string_api.php | 08:19 |
| [KK]Kirill | Merge made by recursive. | 08:19 |
| [KK]Kirill | core/string_api.php | 36 ++++++++++++++++++++++++++++++++++-- | 08:19 |
| [KK]Kirill | 1 files changed, 34 insertions(+), 2 deletions(-) | 08:19 |
| [KK]Kirill | I delete all branch and create new | 08:19 |
| dhx_m | that looks better :) | 08:20 |
| [KK]Kirill | this I can test? | 08:20 |
| mantisbot | New bug: Bug 10360 - karthisudar - open - new | 08:21 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10360 | 08:21 |
| mantisbot | New bug: Bug 10361 - karthisudar - open - new | 08:21 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10361 | 08:21 |
| mantisbot | New bug: Bug 10362 - karthisudar - open - new | 08:21 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10362 | 08:21 |
| mantisbot | New bug: Bug 10363 - karthisudar - open - new | 08:21 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10363 | 08:21 |
| mantisbot | New bug: Bug 10364 - karthisudar - open - new | 08:21 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10364 | 08:21 |
| [KK]Kirill | bun him | 08:21 |
| [KK]Kirill | ban him | 08:21 |
| dhx_m | you can do: "git archive --format=tar mastertest | gzip -9 /some/path/mastertest.tar.gz" to export it | 08:21 |
| dhx_m | oh well, it'll give some devs here a fun job to do later :) | 08:22 |
| [KK]Kirill | :) thanks | 08:23 |
| dhx_m | then you can extract from the archive to test it | 08:25 |
| dhx_m | or you can send the archive to a non-git user to test out | 08:25 |
| mantisbot | New bug: Bug 10365 - karthisudar - open - new | 08:26 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10365 | 08:26 |
| mantisbot | New bug: Bug 10366 - karthisudar - open - new | 08:26 |
| mantisbot | New bug: test - http://www.mantisbt.org/bugs/view.php?id=10366 | 08:26 |
| dhx_m | what could you possibly need to test in 15 bug reports? | 08:26 |
| [KK]Kirill | I can. | 08:28 |
| [KK]Kirill | I just upload string_api.php | 08:32 |
| dhx_m | ah, straight copy from your working directory? | 08:32 |
| dhx_m | to your test site? | 08:32 |
| [KK]Kirill | not to work site | 08:32 |
| dhx_m | or did you push the whole thing to a repo? | 08:33 |
| dhx_m | (was wondering if there is anything to look at on mantisforge) :) | 08:33 |
| [KK]Kirill | I has'n projects on mantisforge | 08:33 |
| [KK]Kirill | I have support site on my work | 08:34 |
| dhx_m | ah ok :) | 08:34 |
| AndyH | Nuclear Eclipse, I found the option to start my bug ID's from say more than 100+, using myphpadmin fine, thanks for the advice, | 08:37 |
| AndyH | but :( ... I've got another quesion! :P | 08:37 |
| AndyH | Has anyone used Mantis with Microsoft SQL server? I'm having issueing with the admin username and password, as mine arn't being acepted | 08:38 |
| dhx_m | ah common... having bug id's < 100 is just a sign that your developers are experts and never make mistakes :p | 08:38 |
| AndyH | i've tried formatting my username into DOMAIN\USERNAME or just USERNAME but neither work. Any ideas? | 08:38 |
| dhx_m | ... can't help there sorry | 08:39 |
| AndyH | Lol, no, that wasn't the intention. I was trying to start my Bug ID's FROM 1000, as we've already used another bit of software to log <1000 | 08:39 |
| AndyH | would be really nice if we only found <100 :) | 08:39 |
| AndyH | Any one else used or have any ideas on using Microsoft SQL server with autenticating usernames and passwords ??? --- ??? | 08:40 |
| * giallu stays away from MS stuff | 08:44 | |
| dhx_m | Mantis uses AdoDB as a backend for all database access | 08:44 |
| dhx_m | so I assume there must be some AdoDB documentation on the topic...? | 08:45 |
| dhx_m | like giallu said, most Mantis developers probably stay a long way away from MSSQL :) | 08:45 |
| giallu | dhx_m, IIRC paulr made some fixes for MSSQL | 08:46 |
| dhx_m | :o so someone does use it | 08:46 |
| giallu | but he tends to work just on trunk | 08:46 |
| dhx_m | lol yeah | 08:46 |
| dhx_m | are you much of a git expert? | 08:46 |
| giallu | so if 1.1.7 is broken on MSSQL you will probably need to test 1.2 | 08:47 |
| dhx_m | I was trying to merge my 10274 branch into a copy of the current master | 08:47 |
| dhx_m | and had a merge conflict... which is easy to fix | 08:47 |
| dhx_m | and git rebase is probably not a good thing if anyone has been using my branch (it is publicly available) | 08:47 |
| dhx_m | however, I assume in most projects, you want to be able to say "please pull from ...." without the other person having to resolve conflicts | 08:49 |
| dhx_m | so now I have the dilemma of "do I rebase or not" :) | 08:49 |
| dhx_m | actually rebasing should be ok... it just means downstream people have to also rebase | 08:51 |
| giallu | dhx_m, we already got bitten git git rebase... | 08:52 |
| giallu | bitten by | 08:53 |
| dhx_m | yeah it is bad if others are depending on your branch | 08:53 |
| dhx_m | (in my case I'm pretty sure no one has extended my branch) | 08:54 |
| giallu | anyway, being that _your_ own branch, feel free to decide... | 09:18 |
| [KK]Kirill | dhx_m: what you want for me testing? | 09:19 |
| dhx_m | 10330 :) | 09:27 |
| dhx_m | I think paul gave it a look over | 09:27 |
| dhx_m | and noted that there needs to be some upgrade path for configuration changes? | 09:27 |
| dhx_m | ie. renaming some constants | 09:27 |
| dhx_m | not really sure if that is possible though | 09:27 |
| dhx_m | giallu: it is more a case of not wanting to make a branch that is impossible/too hard to merge into the mantisbt master | 09:29 |
| dhx_m | I tried merging master into my branch to resolve the conflict... see http://git.mantisforge.org/w/mantisbt/dhx.git?a=shortlog;h=refs/heads/10274 | 09:30 |
| dhx_m | but now it is hard to actually review what changed? | 09:30 |
| mantisbot | New bug: Bug 10367 - cas - open - new | 09:31 |
| mantisbot | New bug: Moving attachments while cloning issue - http://www.mantisbt.org/bugs/view.php?id=10367 | 09:31 |
| mantisbot | New bug: Bug 10368 - jonathh - open - new | 09:31 |
| mantisbot | New bug: Overall Roadmap? - http://www.mantisbt.org/bugs/view.php?id=10368 | 09:31 |
| giallu | dhx_m, we probably need to find a smart way to warn the user the old constant names are deprecated | 09:34 |
| dhx_m | when the user tries to use a new version of Mantis against an old database | 09:35 |
| dhx_m | it takes them to an upgrade screen doesn't it? | 09:35 |
| dhx_m | assuming of course that you get that far without PHP errors warning you that config_inc.php is referencing undefined constants | 09:35 |
| dhx_m | ok it seems my merging went OK: http://git.mantisforge.org/w/mantisbt/dhx.git?a=shortlog;h=refs/heads/fyp | 09:48 |
| dhx_m | ah.... git cherry-pick would have probably been a better choice | 10:04 |
| [KK]Kirill | I need help | 11:45 |
| [KK]Kirill | who around? | 11:45 |
| [KK]Kirill | paul, you here? | 11:48 |
| [KK]Kirill | giallu: dhx_m: you here? | 11:50 |
| dhx_m | yep | 11:59 |
| [KK]Kirill | dhx_m: I have trouble with verify link | 12:04 |
| [KK]Kirill | I switch off self-plugins | 12:05 |
| dhx_m | I'm confused with what you're describing? | 12:05 |
| [KK]Kirill | I switch-on MantisCore Formating | 12:05 |
| [KK]Kirill | but I recieve this mail | 12:05 |
| [KK]Kirill | http://localhost/bug/verify.php?id=29&confirm_hash=a1496b3b227859c4da7a1c8165df6e14 | 12:06 |
| [KK]Kirill | Username: Test | 12:06 |
| [KK]Kirill | Remote IP address: 10.0.128.169 | 12:06 |
| [KK]Kirill | DO NOT REPLY TO THIS MESSAGE | 12:06 |
| [KK]Kirill | http://localhost/bug/verify.php?id=29&confirm_hash=a1496b3b227859c4da7a1c8165df6e14 | 12:07 |
| [KK]Kirill | Username: Test | 12:07 |
| [KK]Kirill | Remote IP address: 10.0.128.169 | 12:07 |
| [KK]Kirill | DO NOT REPLY TO THIS MESSAGE | 12:07 |
| [KK]Kirill | I want recieve last version | 12:07 |
| [KK]Kirill | but html character not transform | 12:07 |
| dhx_m | I know some fixes were made to that stuff recently... might be best to ask paul? | 12:08 |
| [KK]Kirill | I see this task, but if I understand they have error with \n -line separator | 12:09 |
| nuclear_eclipse | morning all | 12:40 |
| dhx_m | hi | 12:41 |
| [KK]Kirill | hi John | 12:45 |
| [KK]Kirill | Help me, John | 12:45 |
| [KK]Kirill | :( | 12:47 |
| [KK]Kirill | daryn: hi | 12:58 |
| daryn | hello | 12:59 |
| [KK]Kirill | John, bug 10369 | 13:03 |
| mantisbot | Bug 10369 - Kirill - open - new | 13:03 |
| mantisbot | Verification link not work - http://www.mantisbt.org/bugs/view.php?id=10369 | 13:03 |
| [KK]Kirill | nuclear_eclipse: Thanks | 13:07 |
| mantisbot | New bug: Bug 10369 - Kirill - open - assigned | 13:07 |
| mantisbot | New bug: Verification link not work - http://www.mantisbt.org/bugs/view.php?id=10369 | 13:07 |
| mantisbot | New bug: Bug 10370 - jonathh - open - new | 13:47 |
| mantisbot | New bug: Max field length not respected. - http://www.mantisbt.org/bugs/view.php?id=10370 | 13:47 |
| nowotny | hello... I noticed mantis supports some html tags in bugnotes but not all... is there a list what it does and does not support somewhere...? | 14:30 |
| giallu | nowotny, yes, in config_defaults_inc.php | 14:32 |
| nowotny | I mean... from the user's point of view... | 14:32 |
| giallu | you can select what to allow and what not | 14:32 |
| giallu | the admin choose what's allowed | 14:33 |
| nowotny | but there are some defaults, right...? | 14:33 |
| giallu | right, in config_defaults_inc.php | 14:33 |
| paul_ | lo | 17:36 |
| nuclear_eclipse | howdy | 17:36 |
| paul_ | howdy | 17:37 |
| paul_ | i'm surprised you say hi to me :) | 17:39 |
| nuclear_eclipse | well, I am rather bored :P | 17:39 |
| paul_ | oh? | 17:39 |
| nuclear_eclipse | bored != not busy... | 17:40 |
| paul_ | wanna discuss subprojects? :) | 17:40 |
| nuclear_eclipse | perhaps | 17:41 |
| paul_ | we seem to do big unncessary loops | 17:41 |
| nuclear_eclipse | probably | 17:41 |
| paul_ | I started looking at caching | 17:42 |
| nuclear_eclipse | I remember being put off by the general level of complexity/redundancy when I was working near it, but didn't feel like touching it more than I already had to | 17:42 |
| paul_ | but decided all i'm doing there is caching big loops | 17:42 |
| paul_ | I want to improve projects for atif ;P | 17:42 |
| paul_ | although | 17:42 |
| paul_ | he seems to have gone quiet and disappeared | 17:43 |
| * daryn_away is away: Gone away for now | 17:46 | |
| mantisbot | New bug: Bug 10371 - kec161 - open - new | 18:24 |
| mantisbot | New bug: Added ability for reminders to come from the user who sent them rather than the from_email - http://www.mantisbt.org/bugs/view.php?id=10371 | 18:24 |
| EduardR | Greetings - checked docs and forums and can't tell whether it's ok to use Postgres (marked experimental). Does it work fine or stick with MySQL for now? Thanks. | 18:40 |
| paul_ | mysql is less hassle | 18:41 |
| CIA-11 | Mantisbt: jreese * rbf6e5ef1cb08 / (config_defaults_inc.php core/print_api.php): Support attached text files with no file extension. | 18:42 |
| paul_ | jreese: we shouldn't expand files with no extension by default | 18:42 |
| paul_ | nuclear_eclipse: even | 18:43 |
| nuclear_eclipse | why not? I've never seen a binary file with no extension, but *lot's* of people use text files with no extension, and it would greatly simplify previewing of patch files on the tracker... | 18:44 |
| * nuclear_eclipse points at bug 10371 | 18:44 | |
| mantisbot | Bug 10371 - kec161 - open - new | 18:44 |
| mantisbot | Added ability for reminders to come from the user who sent them rather than the from_email - http://www.mantisbt.org/bugs/view.php?id=10371 | 18:44 |
| * paul_ nods | 18:45 | |
| nuclear_eclipse | two patches attached, neither with an extension, and I've seen that happen many many times | 18:45 |
| paul_ | but at the same time | 18:45 |
| nuclear_eclipse | and I hate having to save the files to my system just to glance at them... | 18:45 |
| paul_ | if I do git bundle > bundle | 18:45 |
| paul_ | and upload that | 18:45 |
| paul_ | you've got a binary file that's gonna get output to html | 18:46 |
| nuclear_eclipse | ytf would you upload a bundle rather than a set of formatted patches? | 18:46 |
| paul_ | tis an example | 18:46 |
| nuclear_eclipse | and how is that any different than a user renaming an image file to .txt and uploading that? | 18:46 |
| paul_ | true | 18:46 |
| paul_ | but with .txt you know what you are expecting by default :) | 18:46 |
| nuclear_eclipse | either way, you get a whole bunch of gibberish, and at least you can preview it and realize that it's not a text file... | 18:47 |
| paul_ | true | 18:47 |
| nuclear_eclipse | it's not a security risk to preview a binary file, just ugly | 18:47 |
| paul_ | we seem to be starting to get problem reports about plugins into tracker ;/ | 18:48 |
| nuclear_eclipse | I know... | 18:48 |
| nuclear_eclipse | and they aren't even plugins we use... | 18:48 |
| nuclear_eclipse | I could understand getting reports about something like the twitter or source integration plugins, but for something that's obviously barely even supported by the author, I'm not sure why people would ever post bugs against them on our tracker... | 18:49 |
| mantisbot | New bug: Bug 10372 - kec161 - open - new | 18:49 |
| mantisbot | New bug: Don't allow reminders to be sent if the user doesn't have an email address specificed - http://www.mantisbt.org/bugs/view.php?id=10372 | 18:49 |
| nuclear_eclipse | uh, do we even allow users to not specify an email address? | 18:50 |
| paul_ | :) | 18:50 |
| paul_ | hmm | 18:50 |
| paul_ | lets ses | 18:50 |
| nuclear_eclipse | and yth would you want to throw an error because the user's email isn't set? | 18:51 |
| nuclear_eclipse | it would be far better to just silently ignore the problem and continue sending emails... | 18:51 |
| paul_ | anyway | 18:51 |
| paul_ | ignore those | 18:51 |
| nuclear_eclipse | yep | 18:51 |
| paul_ | we need to think about projects | 18:51 |
| nuclear_eclipse | think about it quickly, I need to leave for class in a little bit | 18:52 |
| paul_ | we have: | 18:53 |
| paul_ | project_hierarchy_cache -> which seems to select from project+hierachy table, and store a cache of project_hierarchy + project_inheritance id's | 18:53 |
| paul_ | we have project_hierarchy_get_all_subprojects which seems to get nested subprojects all the way down | 18:54 |
| paul_ | and project_hierarchy_get_subprojects which gets first level of subprojects? | 18:54 |
| nuclear_eclipse | that sounds about right | 18:55 |
| paul_ | then we have | 18:55 |
| paul_ | user_get_accessible_projects, which stores a cache of accessible projects and either: | 18:55 |
| paul_ | a) if someone has private_project_threshold globally, calls project_hierarchy_get_subprojects | 18:56 |
| paul_ | (shouldn't that be _get_all?) | 18:56 |
| * paul_ shrugs | 18:56 | |
| paul_ | or | 18:56 |
| paul_ | b) does it's own select of hierachy+project+project_user_list table | 18:56 |
| nuclear_eclipse | I think the subproject aggregating is a total mess at the moment, and should be taken out back and beaten, and then replaced with something simpler and clearer | 18:57 |
| paul_ | but looking for PUBLIC || PRIVATE+user is on project_user_list table | 18:57 |
| nuclear_eclipse | however, seeing that it actually works right now, I'd rather focus attention elsewhere | 18:58 |
| nuclear_eclipse | quite frankly, I feel that Atif is most likely insane for devising a 30,000 project hierarchy.... | 18:58 |
| paul_ | yea | 18:59 |
| paul_ | at the same time | 18:59 |
| paul_ | what we do atm is a complete mess | 18:59 |
| nuclear_eclipse | agreed, but it still works, which is more than we can say for some other pieces of the app... :P | 18:59 |
| paul_ | function user_get_all_accessible_projects( $p_user_id, $p_project_id ) | 18:59 |
| paul_ | unction user_get_all_accessible_subprojects( $p_user_id, $p_project_id ) | 18:59 |
| paul_ | function user_get_accessible_subprojects( $p_user_id, $p_project_id, $p_show_disabled = false ) | 19:00 |
| paul_ | function user_get_accessible_projects( $p_user_id, $p_show_disabled = false ) | 19:00 |
| paul_ | it's just confusing :) | 19:00 |
| paul_ | we could fix 'atif's issue quite easily: | 19:00 |
| paul_ | by loading hierarchy table on it's own | 19:01 |
| thraxisp | The whole implementation of project based rights has some big holes anyways. | 19:05 |
| thraxisp | paul_: The functions you listed are just variants of each other. | 19:06 |
| nuclear_eclipse | dhx_m: there? | 19:06 |
| paul_ | thraxisp: with various levels of caching | 19:06 |
| paul_ | (different caching) | 19:06 |
| paul_ | we cache the hierachy | 19:07 |
| paul_ | but then dont use it ;/ | 19:07 |
| * paul_ thinks he might try to reorganise it ;/ | 19:10 | |
| * nuclear_eclipse is off to class | 19:19 | |
| nuclear_eclipse | cheers all | 19:19 |
| thraxisp | Doesn't MySQL do caching anyways? Most of our caching seems to be "read the table, and avoid queries". | 19:20 |
| paul_ | well, we dont just support mysql but: | 19:21 |
| paul_ | we try not to read the same data >1 | 19:21 |
| paul_ | it's likely a db query is going to be slower then doing some maths in php | 19:21 |
| w00ter | How do i put the options for status_enum_string in the array field of Configuration Report (which on file looks like; $s_status_enum_string = '10:Gelogd,20:Geprioriteerd,30:Impact Analyse,40:Bouw,50:SRT Test,60:GAT,80:Op Productie,90:Gesloten'; ) | 19:55 |
| dhx_m | hi nuclear_eclipse | 22:49 |
| dhx_m | the import/export plugin throws up this error: | 23:26 |
| dhx_m | APPLICATION WARNING #300: String "manage_import_issues_link" not found. | 23:26 |
| dhx_m | plugin.php?page=XmlImportExport%2Fimport | 23:26 |
Generated by irclog2html.py