xoops forums

scasmar

Just popping in
Posted on: 2012/7/15 13:21
scasmar
scasmar (Show more)
Just popping in
Posts: 78
Since: 2009/7/29
#11

Re: XOOPS Poll 1.32 RC ready for testing

In the link you put CESAG, just download version 1.31 is that correct or am I doing something wrong.

www.funcionariosunidos.es my site I have installed the module and gives me error what exactly is the last version that works properly?

A greeting and thank you very much.

Cesagonchu

Moderator
Posted on: 2012/7/15 22:32
Cesagonchu
Cesagonchu (Show more)
Moderator
Posts: 817
Since: 2010/2/1 1
#12

Re: XOOPS Poll 1.32 RC ready for testing

No, the right link is this.

Sorry, I corrected the link above. I do not know what happened.

DCrussader

Friend of XOOPS
Posted on: 2012/8/7 9:08
DCrussader
DCrussader (Show more)
Friend of XOOPS
Posts: 573
Since: 2005/7/4 3
#13

Re: XOOPS Poll 1.32 RC ready for testing

Updated English language files
http://www.uploading.to/yprcseymnt97
(Someone with a lot of free time and nothing else to do, was playing with pseudo formatting in language files, don't do it again)
Remove outdated translations, use listed here. There are no more such things as language_iso and language_utf8, all translations are utf8.

TX Task - https://www.transifex.com/projects/p/x ... mods-i18n/r/xoopspoll133/
In addition to previously added translations, 3 new

fr-FR http://www.uploading.to/p7ujis5rfd6v
it-IT http://www.uploading.to/ngwjsozqnz28
sv-SE http://www.uploading.to/38fqp1103v0r

irmtfan

Module Developer
Posted on: 2012/9/10 2:10
irmtfan
irmtfan (Show more)
Module Developer
Posts: 3419
Since: 2003/12/7
#14

Re: XOOPS Poll 1.32 RC ready for testing

big issue in xoopspoll 1.33 final version.
In update from a previous version it fail to update the xoopspoll_desc table.
at least one field called anonymous is added in some new version.
1- you should find in which version of xoopspoll the developer add this field.
2- you should add this field.
Error message: Unknown column 'anonymous' in 'field list'
This is really important because xoopspoll is a very old module and almost everybody need to update from an old one.
Also it is very strange to me that nobody notice this issue before.
(i found it in the newbb tests)

I also noticed almost all wishcraft original and improved modules did not tested and are dangerous to use in production websites.

DCrussader

Friend of XOOPS
Posted on: 2012/9/10 6:11
DCrussader
DCrussader (Show more)
Friend of XOOPS
Posts: 573
Since: 2005/7/4 3
#15

Re: XOOPS Poll 1.32 RC ready for testing

Quote:

I also noticed almost all wishcraft original and improved modules did not tested and are dangerous to use in production websites.


Until Cerberus is hired/placed on this position will be like this, even after tests and bug reports, he remain silent - The only human which don't make mistakes during codding, only users do mistakes....

I've already gave up from his modules and tomorrow will start with removal from TX, thanks for the reminder :).

Mamba

Moderator
Posted on: 2012/12/23 5:24
Mamba
Mamba (Show more)
Moderator
Posts: 10333
Since: 2004/4/23
#16

Re: XOOPS Poll 1.32 RC ready for testing

Quote:
In update from a previous version it fail to update the xoopspoll_desc table.
at least one field called anonymous is added in some new version.
1- you should find in which version of xoopspoll the developer add this field.
2- you should add this field.
Error message: Unknown column 'anonymous' in 'field list'

This has been now fixed in SVN
Please support XOOPS & DONATE
Use 2.5.8 | Debug | Requests | Bugs

irmtfan

Module Developer
Posted on: 2012/12/24 0:08
irmtfan
irmtfan (Show more)
Module Developer
Posts: 3419
Since: 2003/12/7
#17

Re: XOOPS Poll 1.32 RC ready for testing

Thank you Mamba
I update the old version to 1.33 final and the field will be added.

Mamba

Moderator
Posted on: 2012/12/24 1:08
Mamba
Mamba (Show more)
Moderator
Posts: 10333
Since: 2004/4/23
#18

Re: XOOPS Poll 1.32 RC ready for testing

Excellent! Thanks for the testing!
Please support XOOPS & DONATE
Use 2.5.8 | Debug | Requests | Bugs

irmtfan

Module Developer
Posted on: 2012/12/25 4:31
irmtfan
irmtfan (Show more)
Module Developer
Posts: 3419
Since: 2003/12/7
#19

Re: XOOPS Poll 1.32 RC ready for testing

I commit persian language to trunk.
It would be good to add local function to this version.
in xoopspoll/admin/xoopspoll.php
change all strtotime:
strtotime($end_time)


with this:
method_exists('XoopsLocal''strtotime') ? XoopsLocal::strtotime($end_time) : strtotime($end_time)


i can add this file to the svn too.

Also IMO it would be better to change the default setting for anonymous in xoopspoll/admin/xoopspoll.php from "1" to "0" to make it similar to the default value in db field:

$anonymous_yn = new XoopsFormRadioYN(_AM_ALLOWANONYMOUS"anonymous"1);


it is better to be 0 because the default value in db is 0

I remember that some people in the past hack this module and add some features like "blind voting". it would be good to add them in future.

edit:
i correct some bugs in newbb regarding xoopspoll 1.33
of course newbb will still work with prev xoopspoll versions like 1.1