| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 09447 | Bug reports | Survey participants (Tokens) | public | 2015-01-09 11:52 | 2015-06-12 12:41 |
| Reporter | DenisChenu | Assigned To | aesteban | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Product Version | 2.05+ | ||||
| Summary | 09447: Token sending email is unusuable with a lot of token | ||||
| Description | With a lot of token, you see a lit of all tokan, the page start by a big list of number. This make the page very long ... long .... and again after click on next. | ||||
| Additional Information | This big list is uneresting : maybe we can hide it completely. Minimu is a max heigth. | ||||
| Tags | No tags attached. | ||||
| Bug heat | 6 | ||||
| Complete LimeSurvey version number (& build) | 150109 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | not relevant | ||||
| Database type & version | not relevant | ||||
| Server OS (if known) | not relevant | ||||
| Webserver software & version (if known) | not relevant | ||||
| PHP Version | not relevant | ||||
 |
@aesteban : i think a global class for this type of "big list" : like the one after sending : just a "max-heigth" and an overflow-y:auto :) |
 |
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=14881 |
|
|
@DenisChenu: I hope you like this approach, this is a standard way to show very long integer lists: 1, 4, 5, 6, 9, 10, 11, 12, 13 => 1, 4-6, 10-13 Unable to merge into 2.06 because 09483 (I add this task to my TODO list) |
|
|
Hi, For 09483 : look at https://bugs.limesurvey.org/view.php?id=9468 think it's same issue here. string(X). This must be fixed in last GIT of 2.06 ? For the fix : i like the idea of 1-10. Why not in viewHelper for 2.06 ? |
|
|
@DenisChenu: Ok, I will put the code in viewHelper for 2.06. By the way: I changed status of 09468. I just reproduced the bug. |
|
|
Fix committed to 2.06 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=14901 |
|
|
I put the code in common_helper.php because viewHelper class is not loaded from views/admin/token/remind.php and invite.php |
 |
Version 2.05 Build 150211 released |
|
|
Still some issue. 1st : see the screenshot, even with this system : it's too big |
|
|
I'm going to replace implode by short_implode in emailpost.php and I'm going to create a new configuration param 'maxstringlengthshortimplode' to choose max string length. If this limit is exceeded, string ends with "..." (see short_implode.png) |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=15028 |
|
|
Fix committed to 2.06 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=15029 |
|
|
2.06 Build 150612 released |
