|Anonymous | Login | Signup for a new account||2021-05-17 23:35 BST|
|Main | My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001107||SiT!||incidents||public||2010-02-25 12:33||2011-02-14 13:18|
|Target Version||3.90beta1||Fixed in Version||Current GIT|
|Summary||0001107: The funtion "autocomplete()" does not work in incident_email page|
|Description||When trying to add some autocomplete functionality to the "incident_email.php" script i noticed that the $pagescripts variable is not treated by the include file "incident_html_top.inc.php". this means that we cannot use autoccomplete in any of the pages that use this header.|
The script that includes the code for the autocreate to work is inside "htmlheader.inc.php" in line 87. Which means that autocomplete can only work when the normal html header is included.
I am not sure what the best solution is here, but i think we could included the same code into "incident_html_top.inc.php"?
|Tags||No tags attached.|
I propose that we add the following code to "incident_html_top.inc.php" at line 49:
// To include a script for a single page, add the filename to the $pagescripts variable before including htmlheader.inc.php
foreach ($pagescripts AS $pscript)
This is the same code as is currently in "htmlheader.inc.php" in line 87.
Let me know what you guys think and i can commit it to svn.
I'm not sure about adding this to 3.60 as this is intended to be a bug fix release only and I'm not convinced this is a bug,
I tend to agree with you, but i was not sure ... on the other hand as 3.6LTS will be long term it may be a while before we can add small things like plugin hooks etc ... For bigger changes i agree with you completely.
We are in the process of doing a whole bunch of small plugins and it would help greatly if we could add for instance, a few plugin hooks and this code proposed here. For us it is a bug, because the function 'autocomplete' does not work in all pages ;-)
|Nope, no new features in 3.60. Sorry to be harsh, but if we open the gates a flood may result.|
|Also, I'm being even more harsh now, and I'm sorry!!! But please don't confirm your own bug reports, the confirmation step provides useful peer review.|
Ivan, thats not harsh just a bit of justice.
I apologise for it seeming that i wanted to "skip" steps .. it was not done on purpose, just because i was in a rush and did not realise what i had selected
I put it here exactly because i was seeking your valuable peer review, before continuing...
and on a second point i apologise for putting 3.60LTS as the target version. I did not realise(not reading i guess) that features were blocked for 3.60.
I will pay closer attention next time .. and sorry if i offended anyone ;-)
|Fixed in commit e9e8fe1|
|2010-02-25 12:33||nicdev||New Issue|
|2010-02-25 12:38||nicdev||Status||new => confirmed|
|2010-02-25 12:42||nicdev||Note Added: 0002470|
|2010-02-25 13:42||paulh||Note Added: 0002471|
|2010-02-25 15:17||nicdev||Note Added: 0002472|
|2010-02-25 16:12||ivan||Note Added: 0002473|
|2010-02-25 16:12||ivan||Target Version||3.60 LTS => 4.0|
|2010-02-25 16:13||ivan||Note Added: 0002474|
|2010-02-25 17:28||nicdev||Note Added: 0002475|
|2010-04-09 14:01||nicdev||Status||confirmed => assigned|
|2010-04-09 14:01||nicdev||Assigned To||=> nicdev|
|2010-04-09 14:02||nicdev||Note Added: 0002983|
|2010-04-09 14:02||nicdev||Status||assigned => resolved|
|2010-04-09 14:02||nicdev||Fixed in Version||=> Current GIT|
|2010-04-09 14:02||nicdev||Resolution||open => fixed|
|2011-02-14 13:18||ivan||Fixed in Version||Current GIT => 3.90beta1|
|2011-02-14 13:18||ivan||Fixed in Version||3.90beta1 => Current GIT|
|2011-02-14 13:18||ivan||Target Version||4.0 => 3.90beta1|
|Copyright © 2000 - 2021 MantisBT Team|