Fail to configure

FabriceV - March 25, 2006 - 08:07
Project:Workspace
Version:4.7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Dear Mr,
Using locally with easyphp, when I try to configure the number of displaying items or files, a message appears:
Please enter a numeric value.
Of course, I have entered a numeric value, but does not function.
Sincerely

#1

hutch - April 22, 2006 - 21:52

The attache diff will fix it, it saves ok, but throws up the following:

warning: urlencode() expects parameter 1 to be string, object given in /home/hutch/www/drupal-4.7.0-rc3/includes/common.inc on line 1282.

I chased around common.inc, but could not see where in workspace.module the error was coming from, and as it does save correctly
that will have to do for now.

AttachmentSize
workspace.diff.txt 984 bytes

#2

jvandyk - April 22, 2006 - 22:36
Status:active» fixed

Commited. Thanks.

#3

hutch - April 24, 2006 - 21:15

Another patch.
This is on the original, so it includes the stuff I submitted on 22 April. It fixes the urlencode error by returning the
workspace_configure_submit function properly back to the workspace page, with an informative message.
It also repairs a bug in the paging, which was not setting up $count_sql properly for the first table on the workspace page.
This was done on MySQL - 4.0.24_Debian-10sarge1-log.
I do not know how this will behave on other versions of mysql ATM, if I get a chance I will test on 4.1.11
probably tomorrow although I doubt if there will be any problems.

HTH

AttachmentSize
workspace1.diff.txt 2.22 KB

#4

jvandyk - May 7, 2006 - 19:45

Committed some parts of the patch by hutch.

#5

hutch - May 8, 2006 - 10:50

Many thanks, this is a big improvement.
One small point, the submit button 'Add new item' has no css,
the attached diff fixes this.
This is on
workspace.module,v 1.21.2.7 2006/05/07 19:44:00 jvandyk

HTH

AttachmentSize
workspace.diff_0.txt 298 bytes

#6

Anonymous - May 22, 2006 - 11:02
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.