[prev in list] [next in list] [prev in thread] [next in thread] 

List:       midgard-dev
Subject:    [midgard-dev] [Midgard 1.x 0000035]: style copy shows ERROR: invalid number of parameters
From:       dev () midgard-project ! org
Date:       2001-08-28 10:32:43
[Download RAW message or body]


The following bug has beeen CLOSED
=======================================================================
http://bugs.midgard-project.org/view_bug_page.php?f_id=0000035
=======================================================================
Reporter:        dheine
Handler:         ab
=======================================================================
Project:         Midgard 1.x
Bug ID:             0000035
Category:        php4-module
Reproducibility: always
Severity:        major
Priority:        normal
Status:          closed
=======================================================================
Date Submitted:   08 Dec 00 04:02 CET
Last Modified:    28 Aug 01 12:32 CEST
=======================================================================
Summary:  style copy shows ERROR: invalid number of parameters

Description: 
the php function copy_style only requires the source id. To place the copy of a style \
tree the target id needs to be accepted. \
=======================================================================

-----------------------------------------------------------------------
 ab - 29 Jul 01 20:56 CEST 
-----------------------------------------------------------------------
mgd_copy_style(id) returns a full copy of the style without changes in the style
itself. For changing root style for copied one just update it with different up
field value.

Same touches for snippetdirs.

-----------------------------------------------------------------------
 ab - 29 Jul 01 21:13 CEST 
-----------------------------------------------------------------------
Look for bug http://http://bugs.midgard-project.org/view_bug_advanced_page.php?f_id=02 \
-- it might be source for this one.

-----------------------------------------------------------------------
 dheine - 30 Jul 01 11:11 CEST 
-----------------------------------------------------------------------
to bugnote 29072056: copy action in asgard tree uses two arguments. php function for \
copying pages also handles two arguments. Therefor the fix under 'Additional Info' \
improves MGD_FUNCTIOn(copy_style) for asgard.  It's not a lib-problem because \
mgd_copy_style works fine with the fix.

to bugnote 29072113: By the way copy of single pages seems to work now after I \
exchanged CVS in july. However the filed bug was made due to copy complete \
asgard_roots(!). I will try this again and report results for closing this bug.

-----------------------------------------------------------------------
 ab - 01 Aug 01 18:02 CEST 
-----------------------------------------------------------------------
Could you provide numbers of lines (and component names in Asgard) where this error \
is reported?

-----------------------------------------------------------------------
 dheine - 07 Aug 01 22:45 CEST 
-----------------------------------------------------------------------
> Warning: Wrong parameter count for mgd_copy_style() in page-init on line 53
of /Asgard_root/style

-----------------------------------------------------------------------
 ab - 18 Aug 01 21:53 CEST 
-----------------------------------------------------------------------
I've fixed this in CVS. Could you check it out and give it a run on your system? I'm \
going to fix all of mgd_copy_foo() functions in a week or so and your feedback would \
be valuable.

-----------------------------------------------------------------------
 dheine - 21 Aug 01 11:10 CEST 
-----------------------------------------------------------------------
my comparison of the php function with the CVS release 1.4 is finished. After beeing \
equal my own release already worked due to my proposal under 'Additional \
Information'. Thanks for the fix in the CVS

-----------------------------------------------------------------------
 ab - 28 Aug 01 12:32 CEST 
-----------------------------------------------------------------------
Fix confirmed

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@midgard-project.org
For additional commands, e-mail: dev-help@midgard-project.org


[prev in list] [next in list] [prev in thread] [next in thread]