PATCH: fix undefined value warning
eMPee584 - July 1, 2008 - 14:53
| Project: | modr8 |
| Version: | 6.x-1.0-beta2 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
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!
| Attachment | Size |
|---|---|
| modr8-fix-undefined-value-#value.patch | 440 bytes |
#1
can you please re-post the patch without a # in the name? I cannot open the link because of that.
#2
wow that's true sorry.. *g
#3
Is this going anywhere? Is it in dev? HEAD?
#4
committed - thanks
#5
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/shIFS=''
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
#6
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.
#7
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 ;)
#8
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.
#9
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.
#10
Automatically closed -- issue fixed for two weeks with no activity.