View Issue Details

IDProjectCategoryView StatusLast Update
0004358mantisbtadministrationpublic2004-08-29 01:47
Reporteravizion Assigned Tothraxisp  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Platformi386OSFreeBSDOS Version4.10-p2
Product Versiongit trunk 
Summary0004358: add user broken in cvs head of 12:08 CET
Description

When I try to create a user on manage_user_create_page.php with all fields filled out properly I get:

"APPLICATION ERROR #200

A required parameter to this page was not found.

Please use the "Back" button.....<snip>"

on page manage_user_create.php.

I have not debugged it further - but it's definately reproduceable!

Time is now 12:08 CET.

  • avizion
Steps To Reproduce

Go this way:

<url>/manage_user_page.php
<url>/manage_user_create_page.php

Fill out form... all fields with valid data.

Press "Create user". Above error will occur.

TagsNo tags attached.
Attached Files
lost_email_patch_patch.diff (571 bytes)   
--- manage_user_create.php	2004-08-19 13:33:08.000000000 +0200
+++ ../cvs/manage_user_create.php	2004-08-16 08:01:52.000000000 +0200
@@ -19,10 +19,8 @@
 
 	$f_username			= gpc_get_string( 'username' );
 	$f_realname			= gpc_get_string( 'realname' );
-	if ( OFF == config_get( 'send_reset_password' ) ) {
 	$f_password			= gpc_get_string( 'password' );
 	$f_password_verify	= gpc_get_string( 'password_verify' );
-	}
 	$f_random_password	= gpc_get_bool( 'random_password' );
 	$f_email			= gpc_get_string( 'email' );
 	$f_access_level		= gpc_get_string( 'access_level' );
lost_email_patch_patch.diff (571 bytes)   

Relationships

related to 0004292 closedthraxisp Sign-up sends a password for LDAP 
related to 0000633 closedmasc email lost password page 
child of 0003987 closedvboctor Mantis 0.19.0 Release 

Activities

avizion

avizion

2004-08-19 06:03

reporter   ~0007061

Speaking of creating accounts...

Users can still apply for accounts and these are created correctly.

Gunnar

Gunnar

2004-08-19 06:36

reporter   ~0007062

Had the same problem. The "lost_email_patch" did add an option 'send_reset_password' that is tested in the file manage_user_create_page.php to determine whether you need to specify an initial password or not. But manage_user_create.php does not check the same variable and always expects a password.

I attached a patch.

Cheers

Gunnar

avizion

avizion

2004-08-19 08:19

reporter   ~0007067

Thanks, but that didn't do the trick for me :(

Unfortunately I'm hung up by other stuff so I can't investigate any further atm.

I'm sure it's something trivial - but never the less does it need fixing :)

Thanks again for quick response

  • avizion
thraxisp

thraxisp

2004-08-19 10:02

reporter   ~0007068

There appear to be some extraneous parameters for creating user accounts.

masc

masc

2004-08-19 10:19

reporter   ~0007070

For sure it was my fault when I sent the diff files. It's an patch integration issue. Let me check the original source code of the patch.

thraxisp

thraxisp

2004-08-19 15:47

reporter   ~0007075

I was planning on applying the following patch:


diff -u -r1.19 manage_user_create.php
--- manage_user_create.php 14 Aug 2004 15:26:20 -0000 1.19
+++ manage_user_create.php 19 Aug 2004 17:35:21 -0000
@@ -19,9 +19,9 @@

    $f_username                     = gpc_get_string( 'username' );
    $f_realname                     = gpc_get_string( 'realname' );
  • $f_password = gpc_get_string( 'password' );
  • $f_password_verify = gpc_get_string( 'password_verify' );
  • $f_random_password = gpc_get_bool( 'random_password' );
  • $f_password = gpc_get_string( 'password', '' );
  • $f_password_verify = gpc_get_string( 'password_verify', '' );
  • $f_random_password = gpc_get_bool( 'random_password', false );
    $f_email = gpc_get_string( 'email' );
    $f_access_level = gpc_get_string( 'access_level' );
    $f_protected = gpc_get_bool( 'protected' );
thraxisp

thraxisp

2004-08-20 08:15

reporter   ~0007080

fixed in CVS

avizion

avizion

2004-08-20 10:00

reporter   ~0007082

I don't see the change in CVS HEAD.

The latest I can see is version 1.19 as of:

Sat Aug 14 15:26:20 2004 UTC (5 days, 23 hours ago) by thraxisp

Tell me if I'm wrong on that account (I did "cvs update" at my local box and I browsed sf.net's CVS server "Current tag: HEAD")? I also tried to apply your 3-line patch, but that didn't make any difference. Same error occur:

"APPLICATION ERROR #200
A required parameter to this page was not found."

Thanks in advance for fixing this :)

  • avizion
thraxisp

thraxisp

2004-08-20 10:06

reporter   ~0007083

There seems to be a long lag (hours) between the CVS commit and it showing up in the web front end. The commit made manage_user_create.php v1.20.

thraxisp

thraxisp

2004-08-20 18:19

reporter   ~0007085

available on sf.net now...