phpesp-devel — Discussion of the development of phpESP

Re: [phpesp-dev] ip column addition missing from 1.8.2 to 2.0.2 database upgrade

[Bishop B. <ph...@id...>]

/me says... We use phing for our build manager, and it has a nice =20
DbDeploy task
http://phing.info/docs/guide/current/chapters/appendixes/AppendixC-OptionalT=
asks.html#DbDeployTask

All you have to do is write delta-set files, and the Phing task will =20
figure out your current database level and apply all the unapplied =20
patches.  It's very effective.

I've got a Phing build file (build.xml) for my local development =20
(moves things around for web deployment, lints my changes, runs some =20
smoke tests), which I've attached for reference.  It doesn't do =20
DbDeploy (as that's not the project standard), but it's something you =20
might consider.

bishop

Quoting Matthew Gregg <mat...@gm...>:

> ahhK... Will update the "update" script to include adding that field.
>
> /me longs for Rails migrations.....
>
> On Fri, 2008-01-11 at 13:08 -0500, Bishop Bettini wrote:
>> Matthew, I was upgrading from 1.8.2, not creating from new.  Thus, I
>> only ran the upgrade scripts.  Anyone upgrading would have this issue.
>>   The bug is that the upgrade scripts did not create the ip column.
>> The fix was the simple ALTER TABLE given below.
>>
>> bishop
>>
>> Quoting Matthew Gregg <mat...@gm...>:
>>
>> > In version 2.0.2 see this in mysql_populate:
>> > CREATE TABLE response (
>> >   id              INT UNSIGNED NOT NULL AUTO_INCREMENT,
>> >   survey_id       INT UNSIGNED NOT NULL,
>> >   submitted       TIMESTAMP NOT NULL DEFAULT '0000-00-00 00:00:00',
>> >   complete        ENUM('Y','N') NOT NULL DEFAULT 'N',
>> >   username        CHAR(64),
>> >   ip              CHAR(64),<- IP is here
>> >   PRIMARY KEY (id),
>> >   KEY `survey_id` (`survey_id`)
>> > );
>> >
>> > The IP field is included.  Wonder how yours was missing?
>> >
>> >
>> >
>> > On Wed, 2008-01-09 at 19:35 -0500, Bishop Bettini wrote:
>> >> REPRODUCTION:
>> >> Starting from a 1.8.2 install, apply the following patches from
>> >> 2.0.2 release:
>> >>
>> >> mysql_update-1.8.2h-1.8.2i.sql
>> >> mysql_update-1.8.2k-2.0.0.sql
>> >>
>> >> Then, create a survey, activate it, then take it.  With debugging
>> >> engaged, the following occurs:
>> >> Request failed, please contact the administrator.[ 1054: Unknown
>> >> column 'ip' in 'field list' ]
>> >>
>> >> EXPECTED:
>> >> The survey should save, and no error message should be emitted
>> >>
>> >> FIX:
>> >> Very simple. Execute this in MySQL client:
>> >> ALTER TABLE response ADD COLUMN ip CHAR(64) AFTER username;
>> >>
>> >
>> >
>> > -----------------------------------------------------------------------=
--
>> > Check out the new SourceForge.net Marketplace.
>> > It's the best place to buy or sell services for
>> > just about anything Open Source.
>> >  =20
>> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketpl=
ace
>> > _______________________________________________
>> > phpESP-devel mailing list
>> > php...@li...
>> > https://lists.sourceforge.net/lists/listinfo/phpesp-devel
>> >
>>
>>
>>
>
>



--=20
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

Re: [phpesp-dev] ip column addition missing from 1.8.2 to 2.0.2 database upgrade

[Matthew G. <mat...@gm...>]

ahhK... Will update the "update" script to include adding that field.

/me longs for Rails migrations.....

On Fri, 2008-01-11 at 13:08 -0500, Bishop Bettini wrote:
> Matthew, I was upgrading from 1.8.2, not creating from new.  Thus, I  
> only ran the upgrade scripts.  Anyone upgrading would have this issue.  
>   The bug is that the upgrade scripts did not create the ip column.   
> The fix was the simple ALTER TABLE given below.
> 
> bishop
> 
> Quoting Matthew Gregg <mat...@gm...>:
> 
> > In version 2.0.2 see this in mysql_populate:
> > CREATE TABLE response (
> >   id              INT UNSIGNED NOT NULL AUTO_INCREMENT,
> >   survey_id       INT UNSIGNED NOT NULL,
> >   submitted       TIMESTAMP NOT NULL DEFAULT '0000-00-00 00:00:00',
> >   complete        ENUM('Y','N') NOT NULL DEFAULT 'N',
> >   username        CHAR(64),
> >   ip              CHAR(64),<- IP is here
> >   PRIMARY KEY (id),
> >   KEY `survey_id` (`survey_id`)
> > );
> >
> > The IP field is included.  Wonder how yours was missing?
> >
> >
> >
> > On Wed, 2008-01-09 at 19:35 -0500, Bishop Bettini wrote:
> >> REPRODUCTION:
> >> Starting from a 1.8.2 install, apply the following patches from   
> >> 2.0.2 release:
> >>
> >> mysql_update-1.8.2h-1.8.2i.sql
> >> mysql_update-1.8.2k-2.0.0.sql
> >>
> >> Then, create a survey, activate it, then take it.  With debugging
> >> engaged, the following occurs:
> >> Request failed, please contact the administrator.[ 1054: Unknown
> >> column 'ip' in 'field list' ]
> >>
> >> EXPECTED:
> >> The survey should save, and no error message should be emitted
> >>
> >> FIX:
> >> Very simple. Execute this in MySQL client:
> >> ALTER TABLE response ADD COLUMN ip CHAR(64) AFTER username;
> >>
> >
> >
> > -------------------------------------------------------------------------
> > Check out the new SourceForge.net Marketplace.
> > It's the best place to buy or sell services for
> > just about anything Open Source.
> > http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > _______________________________________________
> > phpESP-devel mailing list
> > php...@li...
> > https://lists.sourceforge.net/lists/listinfo/phpesp-devel
> >
> 
> 
> 

Re: [phpesp-dev] ip column addition missing from 1.8.2 to 2.0.2 database upgrade

[Bishop B. <ph...@id...>]

Matthew, I was upgrading from 1.8.2, not creating from new.  Thus, I =20
only ran the upgrade scripts.  Anyone upgrading would have this issue. =20
  The bug is that the upgrade scripts did not create the ip column.  =20
The fix was the simple ALTER TABLE given below.

bishop

Quoting Matthew Gregg <mat...@gm...>:

> In version 2.0.2 see this in mysql_populate:
> CREATE TABLE response (
>   id              INT UNSIGNED NOT NULL AUTO_INCREMENT,
>   survey_id       INT UNSIGNED NOT NULL,
>   submitted       TIMESTAMP NOT NULL DEFAULT '0000-00-00 00:00:00',
>   complete        ENUM('Y','N') NOT NULL DEFAULT 'N',
>   username        CHAR(64),
>   ip              CHAR(64),<- IP is here
>   PRIMARY KEY (id),
>   KEY `survey_id` (`survey_id`)
> );
>
> The IP field is included.  Wonder how yours was missing?
>
>
>
> On Wed, 2008-01-09 at 19:35 -0500, Bishop Bettini wrote:
>> REPRODUCTION:
>> Starting from a 1.8.2 install, apply the following patches from  =20
>> 2.0.2 release:
>>
>> mysql_update-1.8.2h-1.8.2i.sql
>> mysql_update-1.8.2k-2.0.0.sql
>>
>> Then, create a survey, activate it, then take it.  With debugging
>> engaged, the following occurs:
>> Request failed, please contact the administrator.[ 1054: Unknown
>> column 'ip' in 'field list' ]
>>
>> EXPECTED:
>> The survey should save, and no error message should be emitted
>>
>> FIX:
>> Very simple. Execute this in MySQL client:
>> ALTER TABLE response ADD COLUMN ip CHAR(64) AFTER username;
>>
>
>
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketpla=
ce
> _______________________________________________
> phpESP-devel mailing list
> php...@li...
> https://lists.sourceforge.net/lists/listinfo/phpesp-devel
>



--=20
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

Re: [phpesp-dev] ip column addition missing from 1.8.2 to 2.0.2 database upgrade

[Matthew G. <mat...@gm...>]

In version 2.0.2 see this in mysql_populate:
CREATE TABLE response (
  id              INT UNSIGNED NOT NULL AUTO_INCREMENT,
  survey_id       INT UNSIGNED NOT NULL,
  submitted       TIMESTAMP NOT NULL DEFAULT '0000-00-00 00:00:00',
  complete        ENUM('Y','N') NOT NULL DEFAULT 'N',
  username        CHAR(64),
  ip              CHAR(64),<- IP is here
  PRIMARY KEY (id),
  KEY `survey_id` (`survey_id`)
);

The IP field is included.  Wonder how yours was missing?



On Wed, 2008-01-09 at 19:35 -0500, Bishop Bettini wrote:
> REPRODUCTION:
> Starting from a 1.8.2 install, apply the following patches from 2.0.2 release:
> 
> mysql_update-1.8.2h-1.8.2i.sql
> mysql_update-1.8.2k-2.0.0.sql
> 
> Then, create a survey, activate it, then take it.  With debugging  
> engaged, the following occurs:
> Request failed, please contact the administrator.[ 1054: Unknown  
> column 'ip' in 'field list' ]
> 
> EXPECTED:
> The survey should save, and no error message should be emitted
> 
> FIX:
> Very simple. Execute this in MySQL client:
> ALTER TABLE response ADD COLUMN ip CHAR(64) AFTER username;
> 

[phpesp-dev] patch: Prompt for confirmation, if Javascript available, after clicking the Purge Surveys button

[Bishop B. <ph...@id...>]

The purge surveys button initiates a permanent and irrevocable  
deletion.  The user should be prompted one final time before  
continuing with the requested operation.  Implement a Javascript  
confirmation pop up (using string translation) to query the user to  
continue.

Patch attached. Should work against SVN/trunk just fine.

bishop

-- 
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

[phpesp-dev] design thoughts needed: tracking login failures and abandoned surveys

[Bishop B. <ph...@id...>]

All,

For my surveys, I'm interested in knowing how many people could not =20
log in (thus failing to complete their surveys) as well as how many =20
people have "given up" while filling out their survey and abandoned =20
the process.

I'm thinking of implementing this as follows.  Login failure count =20
will increment each time an authentication failure occurs.  Abandoned =20
survey count will increment each time a user starts a survey, then =20
decrements each time a user submits a valid survey (ie, all conditions =20
on responses are satisfied).  Completed survey count will increment =20
each time a user submits a valid survey.

All of these would be implemented in a single "statistics" table, as follows=
:

CREATE TABLE statistics (
   metric ENUM ('authfail','abandoned','completed') NOT NULL,
   value  INTEGER NOT NULL DEFAULT 0,
   PRIMARY KEY (metric)
);
INSERT INTO statistics VALUES
  ('authfail', 0),
  ('abandoned', 0),
  ('completed', 0)
;

An interface would be provided in the administrator section to review =20
these values and to reset them, as well.

Thoughts?

bishop

--=20
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

Re: [phpesp-dev] design thoughts needed: hidden from view, but automatically populated, response

[Bishop B. <ph...@id...>]

Matthew, I like your idea of a "respondent data" question type, as I =20
think it's more usable.  But let me elaborate my requirement further =20
to explain why I suggested just a "hidden" type with "template =20
variables":

I need to include respondent data as well as what I'll call "survey =20
data".  The survey itself has information associated with it that is =20
independent of the person completing the survey, but tied directly to =20
the survey itself.

For example, person A might have two surveys to complete: one for =20
class A and one for class B.  I want the survey for class A to include =20
this fact as part of the survey data, so that I can cross tabulate to =20
answer questions like:

"How many men from class B answered 'No' to question 2?"

men =3D respondent data
class B =3D survey data
No =3D response data

So I'd like to add two hidden fields, like this:

name =3D qA
type =3D hidden
text =3D $respondent.gender

name =3D qB
type =3D hidden
text =3D classB

So when I get the survey back, I've got that data submitted and =20
everything else works as its supposed to.  This is probably less =20
usable, as it requires knowledge about what "template variables" are =20
available, but I think it's more flexible.  To answer your question, =20
yes, I would think that all respondent columns would be available =20
(including ID and password).


Now, here's one rub, based on that last sentence: it's a security hole =20
to pass back this "hidden" data, because the user can craft it to be =20
"incorrect" for their situation.  For example, Jane Cracker sees that =20
we're passing back gender =3D "F" as part of her survey.  She's devious, =20
so she wants to change that.  She hops over to wget and posts her =20
responses such that gender =3D "M".  Thus, forging her data.

Perhaps a better way to implement this would be:
1. At survey render time, generate a random, whole number integer key.
2. Store the hidden data for the survey into a table with the given key.
3. Render the survey, including only the key as a hidden variable.
4. After post back, lookup the hidden values using the posted key.
5. array_merge the tabled hidden data into $_POST
6. Delete the tabled data


Thoughts?
bishop


Quoting Matthew Gregg <mat...@gm...>:

> I like the idea of a hidden text field question type.
>
> Are you planning on making all respondent data available as in a
> $respondent['foo'] variable?  Wonder if a "respondent data question
> type" with a drop down to select which bit of respondent data you want,
> would be better?
>
> Don't forget to switch from CVS HEAD to SVN trunk.  I've committed
> favicon patch and hopefully the other changes you sent soon.
>
> On Thu, 2008-01-10 at 18:19 -0500, Bishop Bettini wrote:
>> All,
>>
>> Suppose you want to cross-tabulate respondent last name versus a
>> question on your survey, to answer say something like "How many
>> Smith's said their household income was too low?"
>>
>> Right now, you have to ask the user their last name, because the
>> cross-tabulation works against the survey data, not against the
>> respondent data.  You've already got the user's information, so why
>> ask for it?
>>
>> Yes, this example is contrived.  But, this scenario stems from a
>> particular need of mine, where I'm actually augmenting the respondent
>> data with age, gender, ethnicity, etc. and I want to be able to use
>> the built-in cross-tabulation functionality against those metrics:
>> "How many women answered yes to question 1?"
>>
>> The idea I've come up with is to have "hidden" survey fields, similar
>> to hidden input elements in HTML.  The survey would have place holders
>> for the value, but when the respondent logs in, those input elements
>> aren't seen but are populated from a data source.
>>
>> To implement this, I would add a new type to the question type
>> drop-down called "hidden".  The name would be required, as usual.  The
>> text would either static text or a template variable.
>>
>> so, you might have this:
>>
>> Name =3D Q1
>> Type =3D Hidden
>> Text =3D foo
>>
>> Which would render as
>> <input type=3D"hidden" name=3D"Q1" value=3D"foo" />
>>
>> For my particular need, I'd put in template variable aliases (a la
>> Smarty) for respondent, etc. so that data could be accessed:
>>
>> Name =3D Q1
>> Type =3D Hidden
>> Text =3D $respondent.gender
>>
>> Like executing this:
>> <input type=3D"hidden" name=3D"Q1" value=3D"<?=3D $respondent['gender'] ?=
>" />
>>
>> Yielding after rendering, e.g:
>> <input type=3D"hidden" name=3D"Q1" value=3D"M" />
>>
>>
>> Thoughts?
>>
>> bishop
>>
>
>
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketpla=
ce
> _______________________________________________
> phpESP-devel mailing list
> php...@li...
> https://lists.sourceforge.net/lists/listinfo/phpesp-devel
>



--=20
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

Re: [phpesp-dev] design thoughts needed: hidden from view, but automatically populated, response

[Matthew G. <mat...@gm...>]

I like the idea of a hidden text field question type.

Are you planning on making all respondent data available as in a
$respondent['foo'] variable?  Wonder if a "respondent data question
type" with a drop down to select which bit of respondent data you want,
would be better?

Don't forget to switch from CVS HEAD to SVN trunk.  I've committed
favicon patch and hopefully the other changes you sent soon.

On Thu, 2008-01-10 at 18:19 -0500, Bishop Bettini wrote:
> All,
> 
> Suppose you want to cross-tabulate respondent last name versus a  
> question on your survey, to answer say something like "How many  
> Smith's said their household income was too low?"
> 
> Right now, you have to ask the user their last name, because the  
> cross-tabulation works against the survey data, not against the  
> respondent data.  You've already got the user's information, so why  
> ask for it?
> 
> Yes, this example is contrived.  But, this scenario stems from a  
> particular need of mine, where I'm actually augmenting the respondent  
> data with age, gender, ethnicity, etc. and I want to be able to use  
> the built-in cross-tabulation functionality against those metrics:  
> "How many women answered yes to question 1?"
> 
> The idea I've come up with is to have "hidden" survey fields, similar  
> to hidden input elements in HTML.  The survey would have place holders  
> for the value, but when the respondent logs in, those input elements  
> aren't seen but are populated from a data source.
> 
> To implement this, I would add a new type to the question type  
> drop-down called "hidden".  The name would be required, as usual.  The  
> text would either static text or a template variable.
> 
> so, you might have this:
> 
> Name = Q1
> Type = Hidden
> Text = foo
> 
> Which would render as
> <input type="hidden" name="Q1" value="foo" />
> 
> For my particular need, I'd put in template variable aliases (a la  
> Smarty) for respondent, etc. so that data could be accessed:
> 
> Name = Q1
> Type = Hidden
> Text = $respondent.gender
> 
> Like executing this:
> <input type="hidden" name="Q1" value="<?= $respondent['gender'] ?>" />
> 
> Yielding after rendering, e.g:
> <input type="hidden" name="Q1" value="M" />
> 
> 
> Thoughts?
> 
> bishop
> 

[phpesp-dev] design thoughts needed: hidden from view, but automatically populated, response

[Bishop B. <ph...@id...>]

All,

Suppose you want to cross-tabulate respondent last name versus a =20
question on your survey, to answer say something like "How many =20
Smith's said their household income was too low?"

Right now, you have to ask the user their last name, because the =20
cross-tabulation works against the survey data, not against the =20
respondent data.  You've already got the user's information, so why =20
ask for it?

Yes, this example is contrived.  But, this scenario stems from a =20
particular need of mine, where I'm actually augmenting the respondent =20
data with age, gender, ethnicity, etc. and I want to be able to use =20
the built-in cross-tabulation functionality against those metrics: =20
"How many women answered yes to question 1?"

The idea I've come up with is to have "hidden" survey fields, similar =20
to hidden input elements in HTML.  The survey would have place holders =20
for the value, but when the respondent logs in, those input elements =20
aren't seen but are populated from a data source.

To implement this, I would add a new type to the question type =20
drop-down called "hidden".  The name would be required, as usual.  The =20
text would either static text or a template variable.

so, you might have this:

Name =3D Q1
Type =3D Hidden
Text =3D foo

Which would render as
<input type=3D"hidden" name=3D"Q1" value=3D"foo" />

For my particular need, I'd put in template variable aliases (a la =20
Smarty) for respondent, etc. so that data could be accessed:

Name =3D Q1
Type =3D Hidden
Text =3D $respondent.gender

Like executing this:
<input type=3D"hidden" name=3D"Q1" value=3D"<?=3D $respondent['gender'] ?>" =
/>

Yielding after rendering, e.g:
<input type=3D"hidden" name=3D"Q1" value=3D"M" />


Thoughts?

bishop

--=20
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

[phpesp-dev] [ phpesp-Feature Requests-1868765 ] DB Tables naming convention

[SourceForge.net <no...@so...>]

Feature Requests item #1868765, was opened at 2008-01-10 10:40
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=358956&aid=1868765&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: survey format
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: DB Tables naming convention

Initial Comment:
Add phpesp_ to all of the tables so I can install this into an existing database and ensure I won't have any conflicting tables already in the table.  My Provider only provides one mysql db and I have wordpress and drupal installed in the DB.

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2008-01-10 13:57

Message:
Logged In: NO 

bishop <ph...@id...>

Doesn't the $DB_PREFIX configuration variable address this request?  (Of
course, you have to go through and manually update the SQL to reflect that
prefix.)  Or is the request to come out of the box with a phpesp_ prefix,
so that the end user doesn't have to care?  That sounds like a reasonable
default to me, as people who need it will want it and people who don't need
it won't care.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=358956&aid=1868765&group_id=8956

[phpesp-dev] [ phpesp-Feature Requests-1868765 ] DB Tables naming convention

[SourceForge.net <no...@so...>]

Feature Requests item #1868765, was opened at 2008-01-10 10:40
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=358956&aid=1868765&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: survey format
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: DB Tables naming convention

Initial Comment:
Add phpesp_ to all of the tables so I can install this into an existing database and ensure I won't have any conflicting tables already in the table.  My Provider only provides one mysql db and I have wordpress and drupal installed in the DB.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=358956&aid=1868765&group_id=8956

[phpesp-dev] Moved to SVN completed

[Matthew G. <mat...@gm...>]

No one spoke up so I've made the switch to SVN.  The CVS archive is
still available read-only and all previous CVS people who had CVS commit
should have SVN commit access(yell if you don't).  Info for SVN is here:
http://sourceforge.net/svn/?group_id=8956

The 2.* releases didn't have a branch/tag in CVS and since CVS HEAD was
very close(~2 lines) to 2.0.2 I have made a REL_2.0.2 SVN tag of what
was CVS HEAD.  If bugs need to be fixed with 2.0.2, a 2.0.2 branch
should be made and changes applied to that branch(and trunk if
applicable).

Thanks.

ps. Since I did this pretty fast, if someone yells "bring it back", I
can revert to CVS and re-import to SVN.

[phpesp-dev] phpESP Management Interface string should be controlled by message catalog

[Bishop B. <ph...@id...>]

The text "phpESP Management Login"  
(admin/include/lib/espauth-default.inc:214) should be stored in the  
message catalog, so that it can be properly translated.  Update the  
code and the message catalogs to have this information.

Patch attached.  Don't forget to do:
msgfmt messages.mo

for all your message catalogs... and probably restart the web server, too....

bishop
PS: I'm exhausted, so apologies if something is horribly wrong.

-- 
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

[phpesp-dev] ip column addition missing from 1.8.2 to 2.0.2 database upgrade

[Bishop B. <ph...@id...>]

REPRODUCTION:
Starting from a 1.8.2 install, apply the following patches from 2.0.2 release:

mysql_update-1.8.2h-1.8.2i.sql
mysql_update-1.8.2k-2.0.0.sql

Then, create a survey, activate it, then take it.  With debugging  
engaged, the following occurs:
Request failed, please contact the administrator.[ 1054: Unknown  
column 'ip' in 'field list' ]

EXPECTED:
The survey should save, and no error message should be emitted

FIX:
Very simple. Execute this in MySQL client:
ALTER TABLE response ADD COLUMN ip CHAR(64) AFTER username;

-- 
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

[phpesp-dev] Migration from CVS to SVN

[Matthew G. <mat...@gm...>]

Anyone have any issues with migrating over to SVN on Sourceforge?
Everyone that has any CVS checked out changes will need to commit before
we move.

Re: [phpesp-dev] patch to support favorites icon

[Bishop B. <ph...@id...>]

Hi Matthew,

Out of curiosity, how far is HEAD currently off of v2.0.2?

I ask because we are deploying phpESP for a client who has requested numerou=
s
enhancements.  Some of these enhancements are client-specific, others =20
are much more general and are useful for all phpESP users.  I don't =20
want to work against HEAD, as I'm not sure if it's stable enough for =20
production use.  (Incidentally, the time frame to get these changes in =20
for the client is only 2 weeks, so we are making very small iterations =20
against stable code, so that the branch is always stable.)

Thanks,
bishop

Quoting Matthew Gregg <mat...@gm...>:

> Sorry about the home page being out of sync with SF.  I've updated that
> to reflect 2.0.2, but patches are best submitted against CVS HEAD
> instead of released code.  Your patch should apply against CVS easily
> however.  Thanks!
>
>
> On Wed, 2008-01-09 at 14:33 -0500, Bishop Bettini wrote:
>> phpESP administrators might want to have a favorite's icon for their
>> survey pages.  Since this is (unfortunately) controlled outside of
>> style sheets, a separate configuration directive is needed.  Update
>> the configuration file and the HTML rendering portions of phpESP to
>> support a favorites icon.
>>
>> A patch for this is attached.  This is against 1.8.2, which is the
>> most recent on http://www.butterfat.net/wiki/Projects/phpESP/.  While
>> on SourceForge today, I noticed there's all the way up to 2.0.2.  I
>> should probably start working off of that...  FWIW, this patch works
>> against 2.0.2.
>>
>> bishop
>>
>> -------------------------------------------------------------------------
>> Check out the new SourceForge.net Marketplace.
>> It's the best place to buy or sell services for
>> just about anything Open Source.
>> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketpl=
ace
>> _______________________________________________ phpESP-devel  =20
>> mailing list php...@li...  =20
>> https://lists.sourceforge.net/lists/listinfo/phpesp-devel
>
>
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketpla=
ce
> _______________________________________________
> phpESP-devel mailing list
> php...@li...
> https://lists.sourceforge.net/lists/listinfo/phpesp-devel
>



--=20
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

[phpesp-dev] [ phpesp-Patches-1867891 ] Patch to allow favicon

[SourceForge.net <no...@so...>]

Patches item #1867891, was opened at 2008-01-09 14:57
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=308956&aid=1867891&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Matthew Gregg (greggmc)
Assigned to: Matthew Gregg (greggmc)
Summary: Patch to allow favicon

Initial Comment:
See attachment

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=308956&aid=1867891&group_id=8956

Re: [phpesp-dev] patch to support favorites icon

[Matthew G. <mat...@gm...>]

Sorry about the home page being out of sync with SF.  I've updated that
to reflect 2.0.2, but patches are best submitted against CVS HEAD
instead of released code.  Your patch should apply against CVS easily
however.  Thanks!


On Wed, 2008-01-09 at 14:33 -0500, Bishop Bettini wrote:
> phpESP administrators might want to have a favorite's icon for their  
> survey pages.  Since this is (unfortunately) controlled outside of  
> style sheets, a separate configuration directive is needed.  Update  
> the configuration file and the HTML rendering portions of phpESP to  
> support a favorites icon.
> 
> A patch for this is attached.  This is against 1.8.2, which is the  
> most recent on http://www.butterfat.net/wiki/Projects/phpESP/.  While  
> on SourceForge today, I noticed there's all the way up to 2.0.2.  I  
> should probably start working off of that...  FWIW, this patch works  
> against 2.0.2.
> 
> bishop
> 
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> _______________________________________________ phpESP-devel mailing list php...@li... https://lists.sourceforge.net/lists/listinfo/phpesp-devel

[phpesp-dev] patch to support favorites icon

[Bishop B. <ph...@id...>]

phpESP administrators might want to have a favorite's icon for their  
survey pages.  Since this is (unfortunately) controlled outside of  
style sheets, a separate configuration directive is needed.  Update  
the configuration file and the HTML rendering portions of phpESP to  
support a favorites icon.

A patch for this is attached.  This is against 1.8.2, which is the  
most recent on http://www.butterfat.net/wiki/Projects/phpESP/.  While  
on SourceForge today, I noticed there's all the way up to 2.0.2.  I  
should probably start working off of that...  FWIW, this patch works  
against 2.0.2.

bishop

-- 
Bishop Bettini
ideacode, Inc.
(main) +1 919 341 5170 / (fax) +1 919 521 4100

Visit us on the web at:
ideacode.com          Professional software research and development
reviewmysoftware.com  Improve sales! Review your software before you release
bytejar.com           Solutions to those annoying development problems

[phpesp-dev] [ phpesp-Feature Requests-690818 ] LDAP Auth. Respondant Support

[SourceForge.net <no...@so...>]

Feature Requests item #690818, was opened at 2003-02-21 13:49
Message generated for change (Comment added) made by brubakes
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=358956&aid=690818&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Matthew Gregg (greggmc)
Summary: LDAP Auth. Respondant Support

Initial Comment:
I know that the designer accounts can authenticate 
against an LDAP database but we really need to have 
our larger respondant users be able to login using their 
existing LDAP account to respond to a private survey.

----------------------------------------------------------------------

Comment By: brubakes (brubakes)
Date: 2008-01-04 16:12

Message:
Logged In: YES 
user_id=1973659
Originator: NO

I agree.  This would make phpESP so much more usable for us.  I've setup
phpESP to use LDAP for designers and superusers, but cannot get ldap to
work for responders (even with 2.0.2).

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2004-11-02 12:52

Message:
Logged In: NO 

This would be a very nice feature.  Using Ldap against an AD
lookup is what Im looking for.  I might just breakdown and
program it myself :)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=358956&aid=690818&group_id=8956

[phpesp-dev] [ phpesp-Bugs-1856541 ] Syntax error when creating designer

[SourceForge.net <no...@so...>]

Bugs item #1856541, was opened at 2007-12-22 20:37
Message generated for change (Comment added) made by liedekef
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1856541&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Syntax error when creating designer

Initial Comment:
When I try to add a designer at
http://phpesp.mydomain.com/admin/manage.php?where=admdesigner

I get the following error:

Parse error: syntax error, unexpected T_BOOLEAN_AND in /home/homepages/hosts/phpesp.cwittenb.org/admin/include/where/admdesigner.inc on line 16


----------------------------------------------------------------------

>Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-26 18:04

Message:
Logged In: YES 
user_id=109671
Originator: NO

Hi,

this has been fixed in CVS already, see the fix for the file
admin/include/where/admdesigner.inc: 
http://phpesp.cvs.sourceforge.net/phpesp/phpESP/admin/include/where/admdesigner.inc?r1=1.19&r2=1.20

Franky

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1856541&group_id=8956

[phpesp-dev] [ phpesp-Bugs-1856541 ] Syntax error when creating designer

[SourceForge.net <no...@so...>]

Bugs item #1856541, was opened at 2007-12-22 11:37
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1856541&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Syntax error when creating designer

Initial Comment:
When I try to add a designer at
http://phpesp.mydomain.com/admin/manage.php?where=admdesigner

I get the following error:

Parse error: syntax error, unexpected T_BOOLEAN_AND in /home/homepages/hosts/phpesp.cwittenb.org/admin/include/where/admdesigner.inc on line 16


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1856541&group_id=8956

[phpesp-dev] [ phpesp-Bugs-1846497 ] Did Upgrade, Problem when Edit or Adding

[SourceForge.net <no...@so...>]

Bugs item #1846497, was opened at 2007-12-07 22:30
Message generated for change (Comment added) made by liedekef
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1846497&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Admin
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Franky Van Liedekerke (liedekef)
Summary: Did Upgrade, Problem when Edit or Adding

Initial Comment:
I just upgrade from 1.82 to 2.01.  I did the database upgrade.  Saved old version to another folder.  Created same folder name for new version and update the phpESP.ini.php to reflect my old setting.

The program runs, but when I try to add or edit a survey I get this error on the top of the screen->

[ Warning, error encountered. [ 1054: Unknown column 'auto_num' in 'field list' ] ] 


Please advise on a path to take.  Thanks Chuck
cc4...@ya...

----------------------------------------------------------------------

>Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-12 09:40

Message:
Logged In: YES 
user_id=109671
Originator: NO

No problem, glad to be of help.

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-12 09:25

Message:
Logged In: NO 

liedekef:->

Thank you for your help.  I am up and running.  Sorry for the wrong line
edit.  I opened it in NotePad and somehow it added a hard return into the
document.  This time I opened it in phpDesigner.  Made the correct change
as you suggested and program is running correctly.

I look forward to future update as the program keeps getting better and
better.

Thanks for you time.

Chuck


----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-11 11:13

Message:
Logged In: YES 
user_id=109671
Originator: NO

You edited the wrong line ... please do make sure you check these things
before posting.
This is how it should look like (see also CVS):
                if(isset($_POST['edit_question']))
                        $curr_q = $_POST['q_id'];
                if($curr_q<0)
                        $curr_q = 1;
Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 23:55

Message:
Logged In: NO 

Well I made the change you suggested and am still getting the error for
problem 2(The fresh copy)
Here is the changes I made->
Do I need to change line 32 also????
// update failed, stay on same question
	if(!$updated || isset($_POST['extra_choices'])) {
		$curr_q = $id;
	} else {
		if(isset($_POST['new_question']))
			$curr_q = 0;
		if(isset($_POST['edit_question']))
			$curr_q = $_POST['curr_q'];
		if($q_id<0)
			$curr_q = 1;
	}


----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 23:36

Message:
Logged In: NO 

That is OK regarding problem 1.  I figure the problem must be from how I
did the update.  It not a real big problem becuase I only have 1 active
survey anyway which  I can put over on my other server.  Thus, I am going
to work on problem 2 which is a fresh install.  Thus, any problem should be
eaiser to fix.


----------------------------------------------------------------------

Comment By: Matthew Gregg (greggmc)
Date: 2007-12-09 21:49

Message:
Logged In: YES 
user_id=14116
Originator: NO

FYI phpESP should never work with globals on

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 13:11

Message:
Logged In: YES 
user_id=109671
Originator: NO

Sorry, but your error is not reproducible here ... please provide error
logs if any.

Franky

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 13:07

Message:
Logged In: YES 
user_id=109671
Originator: NO

I tested it and it is working just fine. The only difference is that in my
php I don't have register_globals set and also not magic_quotes_gpc.
I'll turn each of them on and try again ...

Franky

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 12:52

Message:
Logged In: YES 
user_id=109671
Originator: NO

Problem 2 is easy: first, it is not an error but a php notice, you
shouldn't be showing this on your screen for security reasons. Anyway, it
shouldn't happen. The fix: change the name of q_id into curr_q on line 31
in admin/include/tab/questions.inc (I'll change this in CVS as well).
Problem 1: still investigating ...

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 03:18

Message:
Logged In: NO 

Got a problem.  Something is not right.
I did the update of 1.82h to 1.82i.  Mysql said all was fine.  Seemed to
work fine.

Problem1
So, I created a survey, tested it.  All seemed to run correctly.  I then
activated it.  All seemed correct.  The problem come when I fill out or
anyone person fills out the survey.  I get the backup email as I should,
but when I go to look at the stat section it shows no surveys summitted.


Problem2 with a newly created version
I then decided it might be easier to just recreate the php program from
scratch.  
Thus on one of my other domains and I setup the  php2 from scratch.
1)I created the database in phpadmin instead of running the
mysql_create.sql script.
2) I ran the mysql_populate.sql script and all ran fine without any
errors.
3) I then copied the php files(leaving the file structure the same) to the
server.
4) I edited the phpESP.ini.php from admin.
5) I ran the test.php from admin.  All seemed ok.  Here is the link if you
would like to
see->http://catradestationug.com/php_scripts/phpESP2/admin/test.php
6) Problem comes when I try to add a question-->>I get this error---
Notice: Undefined variable: q_id in
/home/catrades/public_html/php_scripts/phpESP2/admin/include/tab/questions.inc
on line 31

Any ideas.

Sorry for the issues.

chuck



----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-08 23:33

Message:
Logged In: YES 
user_id=109671
Originator: NO

Yes, do run it, mysql won't mind. The only thing that happens in the file
mysql_update-1.8.2h-1.8.2i.sql is the creation of that specific column.

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-08 20:52

Message:
Logged In: NO 

Yes I read the Update.txt file.  I ran the update 1.82k-2.0.0.

I didn't run the 1.82h to 1.82i.  I know my version is 1.82, but I am not
sure of the letter number.

The 1.82 to 2.0.0 had no errors on the update and completed sucessfully.

Should I try to run the 1.82h- to 1.82i.  I don't think mysql will let me
run it since the database has been updated to 2.0.

Please advise.  Thanks for the response.

Chuck

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-08 16:42

Message:
Logged In: YES 
user_id=109671
Originator: NO

Hi,

did you read the file called "UPDATE"? You forgot to do the DB upgrades.
Go into the dir scripts/db and run the following mysql statements on your
database:
mysql_update-1.8.2h-1.8.2i.sql
and then
mysql_update-1.8.2k-2.0.0.sql

Franky

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1846497&group_id=8956

[phpesp-dev] [ phpesp-Bugs-1846497 ] Did Upgrade, Problem when Edit or Adding

[SourceForge.net <no...@so...>]

Bugs item #1846497, was opened at 2007-12-07 13:30
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1846497&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Admin
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Franky Van Liedekerke (liedekef)
Summary: Did Upgrade, Problem when Edit or Adding

Initial Comment:
I just upgrade from 1.82 to 2.01.  I did the database upgrade.  Saved old version to another folder.  Created same folder name for new version and update the phpESP.ini.php to reflect my old setting.

The program runs, but when I try to add or edit a survey I get this error on the top of the screen->

[ Warning, error encountered. [ 1054: Unknown column 'auto_num' in 'field list' ] ] 


Please advise on a path to take.  Thanks Chuck
cc4...@ya...

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-12 00:25

Message:
Logged In: NO 

liedekef:->

Thank you for your help.  I am up and running.  Sorry for the wrong line
edit.  I opened it in NotePad and somehow it added a hard return into the
document.  This time I opened it in phpDesigner.  Made the correct change
as you suggested and program is running correctly.

I look forward to future update as the program keeps getting better and
better.

Thanks for you time.

Chuck


----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-11 02:13

Message:
Logged In: YES 
user_id=109671
Originator: NO

You edited the wrong line ... please do make sure you check these things
before posting.
This is how it should look like (see also CVS):
                if(isset($_POST['edit_question']))
                        $curr_q = $_POST['q_id'];
                if($curr_q<0)
                        $curr_q = 1;
Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 14:55

Message:
Logged In: NO 

Well I made the change you suggested and am still getting the error for
problem 2(The fresh copy)
Here is the changes I made->
Do I need to change line 32 also????
// update failed, stay on same question
	if(!$updated || isset($_POST['extra_choices'])) {
		$curr_q = $id;
	} else {
		if(isset($_POST['new_question']))
			$curr_q = 0;
		if(isset($_POST['edit_question']))
			$curr_q = $_POST['curr_q'];
		if($q_id<0)
			$curr_q = 1;
	}


----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 14:36

Message:
Logged In: NO 

That is OK regarding problem 1.  I figure the problem must be from how I
did the update.  It not a real big problem becuase I only have 1 active
survey anyway which  I can put over on my other server.  Thus, I am going
to work on problem 2 which is a fresh install.  Thus, any problem should be
eaiser to fix.


----------------------------------------------------------------------

Comment By: Matthew Gregg (greggmc)
Date: 2007-12-09 12:49

Message:
Logged In: YES 
user_id=14116
Originator: NO

FYI phpESP should never work with globals on

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 04:11

Message:
Logged In: YES 
user_id=109671
Originator: NO

Sorry, but your error is not reproducible here ... please provide error
logs if any.

Franky

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 04:07

Message:
Logged In: YES 
user_id=109671
Originator: NO

I tested it and it is working just fine. The only difference is that in my
php I don't have register_globals set and also not magic_quotes_gpc.
I'll turn each of them on and try again ...

Franky

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 03:52

Message:
Logged In: YES 
user_id=109671
Originator: NO

Problem 2 is easy: first, it is not an error but a php notice, you
shouldn't be showing this on your screen for security reasons. Anyway, it
shouldn't happen. The fix: change the name of q_id into curr_q on line 31
in admin/include/tab/questions.inc (I'll change this in CVS as well).
Problem 1: still investigating ...

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-08 18:18

Message:
Logged In: NO 

Got a problem.  Something is not right.
I did the update of 1.82h to 1.82i.  Mysql said all was fine.  Seemed to
work fine.

Problem1
So, I created a survey, tested it.  All seemed to run correctly.  I then
activated it.  All seemed correct.  The problem come when I fill out or
anyone person fills out the survey.  I get the backup email as I should,
but when I go to look at the stat section it shows no surveys summitted.


Problem2 with a newly created version
I then decided it might be easier to just recreate the php program from
scratch.  
Thus on one of my other domains and I setup the  php2 from scratch.
1)I created the database in phpadmin instead of running the
mysql_create.sql script.
2) I ran the mysql_populate.sql script and all ran fine without any
errors.
3) I then copied the php files(leaving the file structure the same) to the
server.
4) I edited the phpESP.ini.php from admin.
5) I ran the test.php from admin.  All seemed ok.  Here is the link if you
would like to
see->http://catradestationug.com/php_scripts/phpESP2/admin/test.php
6) Problem comes when I try to add a question-->>I get this error---
Notice: Undefined variable: q_id in
/home/catrades/public_html/php_scripts/phpESP2/admin/include/tab/questions.inc
on line 31

Any ideas.

Sorry for the issues.

chuck



----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-08 14:33

Message:
Logged In: YES 
user_id=109671
Originator: NO

Yes, do run it, mysql won't mind. The only thing that happens in the file
mysql_update-1.8.2h-1.8.2i.sql is the creation of that specific column.

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-08 11:52

Message:
Logged In: NO 

Yes I read the Update.txt file.  I ran the update 1.82k-2.0.0.

I didn't run the 1.82h to 1.82i.  I know my version is 1.82, but I am not
sure of the letter number.

The 1.82 to 2.0.0 had no errors on the update and completed sucessfully.

Should I try to run the 1.82h- to 1.82i.  I don't think mysql will let me
run it since the database has been updated to 2.0.

Please advise.  Thanks for the response.

Chuck

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-08 07:42

Message:
Logged In: YES 
user_id=109671
Originator: NO

Hi,

did you read the file called "UPDATE"? You forgot to do the DB upgrades.
Go into the dir scripts/db and run the following mysql statements on your
database:
mysql_update-1.8.2h-1.8.2i.sql
and then
mysql_update-1.8.2k-2.0.0.sql

Franky

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1846497&group_id=8956

[phpesp-dev] [ phpesp-Bugs-1846497 ] Did Upgrade, Problem when Edit or Adding

[SourceForge.net <no...@so...>]

Bugs item #1846497, was opened at 2007-12-07 22:30
Message generated for change (Comment added) made by liedekef
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1846497&group_id=8956

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Admin
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Franky Van Liedekerke (liedekef)
Summary: Did Upgrade, Problem when Edit or Adding

Initial Comment:
I just upgrade from 1.82 to 2.01.  I did the database upgrade.  Saved old version to another folder.  Created same folder name for new version and update the phpESP.ini.php to reflect my old setting.

The program runs, but when I try to add or edit a survey I get this error on the top of the screen->

[ Warning, error encountered. [ 1054: Unknown column 'auto_num' in 'field list' ] ] 


Please advise on a path to take.  Thanks Chuck
cc4...@ya...

----------------------------------------------------------------------

>Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-11 11:13

Message:
Logged In: YES 
user_id=109671
Originator: NO

You edited the wrong line ... please do make sure you check these things
before posting.
This is how it should look like (see also CVS):
                if(isset($_POST['edit_question']))
                        $curr_q = $_POST['q_id'];
                if($curr_q<0)
                        $curr_q = 1;
Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 23:55

Message:
Logged In: NO 

Well I made the change you suggested and am still getting the error for
problem 2(The fresh copy)
Here is the changes I made->
Do I need to change line 32 also????
// update failed, stay on same question
	if(!$updated || isset($_POST['extra_choices'])) {
		$curr_q = $id;
	} else {
		if(isset($_POST['new_question']))
			$curr_q = 0;
		if(isset($_POST['edit_question']))
			$curr_q = $_POST['curr_q'];
		if($q_id<0)
			$curr_q = 1;
	}


----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 23:36

Message:
Logged In: NO 

That is OK regarding problem 1.  I figure the problem must be from how I
did the update.  It not a real big problem becuase I only have 1 active
survey anyway which  I can put over on my other server.  Thus, I am going
to work on problem 2 which is a fresh install.  Thus, any problem should be
eaiser to fix.


----------------------------------------------------------------------

Comment By: Matthew Gregg (greggmc)
Date: 2007-12-09 21:49

Message:
Logged In: YES 
user_id=14116
Originator: NO

FYI phpESP should never work with globals on

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 13:11

Message:
Logged In: YES 
user_id=109671
Originator: NO

Sorry, but your error is not reproducible here ... please provide error
logs if any.

Franky

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 13:07

Message:
Logged In: YES 
user_id=109671
Originator: NO

I tested it and it is working just fine. The only difference is that in my
php I don't have register_globals set and also not magic_quotes_gpc.
I'll turn each of them on and try again ...

Franky

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-09 12:52

Message:
Logged In: YES 
user_id=109671
Originator: NO

Problem 2 is easy: first, it is not an error but a php notice, you
shouldn't be showing this on your screen for security reasons. Anyway, it
shouldn't happen. The fix: change the name of q_id into curr_q on line 31
in admin/include/tab/questions.inc (I'll change this in CVS as well).
Problem 1: still investigating ...

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-09 03:18

Message:
Logged In: NO 

Got a problem.  Something is not right.
I did the update of 1.82h to 1.82i.  Mysql said all was fine.  Seemed to
work fine.

Problem1
So, I created a survey, tested it.  All seemed to run correctly.  I then
activated it.  All seemed correct.  The problem come when I fill out or
anyone person fills out the survey.  I get the backup email as I should,
but when I go to look at the stat section it shows no surveys summitted.


Problem2 with a newly created version
I then decided it might be easier to just recreate the php program from
scratch.  
Thus on one of my other domains and I setup the  php2 from scratch.
1)I created the database in phpadmin instead of running the
mysql_create.sql script.
2) I ran the mysql_populate.sql script and all ran fine without any
errors.
3) I then copied the php files(leaving the file structure the same) to the
server.
4) I edited the phpESP.ini.php from admin.
5) I ran the test.php from admin.  All seemed ok.  Here is the link if you
would like to
see->http://catradestationug.com/php_scripts/phpESP2/admin/test.php
6) Problem comes when I try to add a question-->>I get this error---
Notice: Undefined variable: q_id in
/home/catrades/public_html/php_scripts/phpESP2/admin/include/tab/questions.inc
on line 31

Any ideas.

Sorry for the issues.

chuck



----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-08 23:33

Message:
Logged In: YES 
user_id=109671
Originator: NO

Yes, do run it, mysql won't mind. The only thing that happens in the file
mysql_update-1.8.2h-1.8.2i.sql is the creation of that specific column.

Franky

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2007-12-08 20:52

Message:
Logged In: NO 

Yes I read the Update.txt file.  I ran the update 1.82k-2.0.0.

I didn't run the 1.82h to 1.82i.  I know my version is 1.82, but I am not
sure of the letter number.

The 1.82 to 2.0.0 had no errors on the update and completed sucessfully.

Should I try to run the 1.82h- to 1.82i.  I don't think mysql will let me
run it since the database has been updated to 2.0.

Please advise.  Thanks for the response.

Chuck

----------------------------------------------------------------------

Comment By: Franky Van Liedekerke (liedekef)
Date: 2007-12-08 16:42

Message:
Logged In: YES 
user_id=109671
Originator: NO

Hi,

did you read the file called "UPDATE"? You forgot to do the DB upgrades.
Go into the dir scripts/db and run the following mysql statements on your
database:
mysql_update-1.8.2h-1.8.2i.sql
and then
mysql_update-1.8.2k-2.0.0.sql

Franky

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=108956&aid=1846497&group_id=8956