hi there,
attached patch fixes a warning (E_ALL) when you visit the Views module page... thought i better quickly submit this before i forget.. regards!

CommentFileSizeAuthor
#2 modr8-fix-undefined-value-value.patch440 byteseMPee584
modr8-fix-undefined-value-#value.patch440 byteseMPee584

Comments

pwolanin’s picture

pwolanin commented

can you please re-post the patch without a # in the name? I cannot open the link because of that.

eMPee584’s picture

eMPee584 commented
StatusFileSize
new modr8-fix-undefined-value-value.patch440 bytes

wow that's true sorry.. *g

niklp’s picture

niklp commented

Is this going anywhere? Is it in dev? HEAD?

pwolanin’s picture

pwolanin commented
Status: Needs review » Fixed

committed - thanks

eMPee584’s picture

eMPee584 commented

cool thx pwolanin... btw would you mind to also make 6.x-dev releases on each CVS commit? don't know how much work it is but it helps stay on track, f.e. my little update script uses the date of dev tarballs to slurp newest ones off the server..

#!/bin/sh
IFS=''
echo "`date`">update.log
for file in *.tar.gz
do
        cp -p $file $file.tmpcpy
        wget -N --no-proxy http://ftp.drupal.org/files/projects/$file >>update.log 2>&1
        if [[ "$file" -nt "$file.tmpcpy" ]]
                then
                tar -xzf $file
                echo "$file updated"
        fi
        rm $file.tmpcpy
done
niklp’s picture

niklp commented

Again, there are still no clues as to "where this is fixed" - can someone PLEASE add this information? Lots of people are trying their hardest to use 6.x stuff. Modules are all over the place, still, and lack of this kind of information is just confusing as hell.

Many thanks.

eMPee584’s picture

eMPee584 commented

NikLP don't quite now if i understood you correctly, but if you meant that literally, pwolanin has committed this to CVS HEAD, which is the only place to get it because this module does not yet have a dev release. It's only a silent warning anyways so please calm down ;)

pwolanin’s picture

pwolanin commented

eMPee584 - if you want to stay current like that, I'd suggest deploying via CVS.

I'll push a 1.0 with this fix anyhow.

eMPee584’s picture

eMPee584 commented

nah CVS has them stupid hidden folders ;)
but almost all packages i have installed do -dev-releases so except for a few my method works nicely.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.