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

List:       midgard-dev
Subject:    [midgard-dev] Re: [Midgard 1.x 0000035]: style copy shows ERROR: invalid number of
From:       Dieter Heine <dieter () work ! kn-bremen ! de>
Date:       2001-08-19 7:08:20
[Download RAW message or body]

hello alex,

thanks for your fix. 9 hours later I checked it against my homefix from 28.July \
(already working) . Ooops - in the mail below the submition date shows 28.Dec. \
However there was no diff except the function header that you  fixed, too.  Though I \
know it would be better to recompile with your release I would like to wait for a \
recompile with a full CVS (e.g. 1.4.2). Our functions are equal. I guess you took the \
same way and copied the copy_page function with textual replacement of page against \
style.

 However there have not been problems with style copies in asgard since my homefix.

regards,    dieter

dev@midgard-project.org wrote:

> The following bug has been REOPENED.
> =======================================================================
> 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:          feedback
> =======================================================================
> Date Submitted:   08 Dec 00 04:02 CET
> Last Modified:    18 Aug 01 21:53 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.


---------------------------------------------------------------------
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]