Re: [Apc-aa-coders] PS to Broken new_unalias_recurent() in current CVS

[Mitra <mi...@mi...>]

OK - its fixed,

The code in there relied on the bug that 'headline........'  gets 
expanded, when really it should be '{headline........}'

I've put some code into item.php3, to recognize parameters that are 
fields, and expand directly instead of using the new_unalias_recurent.

This fix is checked into CVS, so please update and it should fix it.

- Mitra


>Marek
>
>Since both Honza and I have been working on code around this area, 
>can you make sure that you update to the latest CVS immediately 
>before reporting the bug, then I (or Honza) will know we are 
>duplicating your bug on the same code.
>
>I *am* working most of this weekend, so I should be able to get to 
>this quickly.
>
>- Mitra
>
>>Marek
>>
>>Its perfectly legitimate to not pass parameters, you'll notice that 
>>parameters 6 and 7 contain defaults.
>>
>>For the other item, do you have
>>
>>1: An alias e.g. _#HEAD_URL  defined using this function
>>OR
>>have it bare in a view - e.g. 
>>{alias:headline........:f_b::/members/country/full.shtml:headline........::class="res_link":}
>>
>>Let me know so that I can duplicate the problem.
>>
>>It would also help if you gave me a URL I could test it on - and/or 
>>ran that URL adding "debug=1" to the result and emailed me the 
>>result.
>>
>>- Mitra
>>
>>
>>
>>At 8:10 PM +0100 10/1/03, Marek Tichy wrote:
>>>Just tried cvs update and afterwards got lots of
>>>
>>>Warning: Missing argument 7 for new_unalias_recurent() in 
>>>/data/www/htdocs/ein.org.uk/aa/include/stringexpand.php3 on line 
>>>210
>>>
>>>Warning: Missing argument 6 for new_unalias_recurent() in 
>>>/data/www/htdocs/ein.org.uk/aa/include/stringexpand.php3 on line 
>>>210
>>>
>>>But that's just a minor issue, which can be fixed
>>>
>>>bash-2.05b$ diff item.php3 item.php3-
>>>409c409
>>><     return new_unalias_recurent($text, $remove, $level, 
>>>$maxlevel, $this,null,null); # Note no itemview param
>>>---
>>>  >     return new_unalias_recurent($text, $remove, $level, 
>>>$maxlevel, $this ); # Note no itemview param
>>>
>>>But new_unalias_recurent is not unaliasing properly, i have things 
>>>like headline..... - when printing link using f_b function with
>>>following params:
>>>
>>>::/members/country/full.shtml:headline........::class="res_link":
>>>
>>>all over the site.
>>>
>>>Any ideas ?
>>>
>>>It's likely that the problem is in new_unalias_recurent function, 
>>>but I'm unable to fix.
>>>
>>>Marek
>>>
>>>  > -----Original Message-----
>>>  > From: apc...@li...
>>>  > [mailto:apc...@li...]On Behalf Of Mitra
>>>  > Sent: 03 January 2003 23:20
>>>  > To: APC AA General
>>>  > Subject: [Apc-aa-coders] Remove string behavior - possible change
>>>  >
>>>  >
>>>  > Is anyone likely  to have problems if the Remove strings apply to the
>>>  > entire result of a view/slice rather than just to certain parts as it
>>>  > does now?
>>>  >
>>>  > Honza - I'm ready to CVS submit the merged string handling code, but
>>>  > just need your ok on doing this because of the change to Remove.
>>>  >
>>>  > - Mitra
>>>  >
>>>  > --
>>>  > Mitra Technology Consulting -  www.mitra.biz - mi...@mi...
>>>  > 02-6684-8096  or 0414-648-0722
>>>  >
>>>  > Life is a Mystery to be Lived, not a Problem to be Solved
>>>  >
>>>  >
>>>  >
>>>  > -------------------------------------------------------
>>>  > This sf.net email is sponsored by:ThinkGeek
>>>  > Welcome to geek heaven.
>>>  > http://thinkgeek.com/sf
>>>  > _______________________________________________
>>>  > Apc-aa-coders mailing list
>>>  > Apc...@li...
>>>  > https://lists.sourceforge.net/lists/listinfo/apc-aa-coders
>>>  >
>>>
>>>
>>>
>>>-------------------------------------------------------
>>>This SF.NET email is sponsored by:
>>>SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See!
>>>http://www.vasoftware.com
>>>_______________________________________________
>>>Apc-aa-coders mailing list
>>>Apc...@li...
>>>https://lists.sourceforge.net/lists/listinfo/apc-aa-coders
>>

-- 
Mitra Technology Consulting -  www.mitra.biz - mi...@mi...
02-6684-8096  or 0414-648-0722 

Life is a Mystery to be Lived, not a Problem to be Solved