636 Posts in 190 Topics by 159 members
| Go to End | Next > | |
| Author | Topic: | 2408 Views |
-
a bunch of new modules

8 October 2010 at 6:49pm Last edited: 8 October 2010 6:50pm
Hi
I have been working on a bunch of new modules over the last few months:
* wishlist
* campaign monitor
* vimeo
* etc...see: http://sunny.svnrepository.com/sunny-side-up-general/trac.cgi/browser
Most of them are pre-alpha releases.
Feel free to contact me if you have any questions / worries / contributions.
- Nicolaas
-
Re: a bunch of new modules

12 October 2010 at 8:02am
Wow thats an impressive list of modules! Are you making sure they find their way onto the ss.org/modules listing once they're ready for public usage?
I've noticed that some of them don't have a LICENSE file associated with them, which could prohibit usage.
Also, did you consider using markdown for formatting the README? It makes it much more readable,
even when there's no markdown->html formatter applied. Have a look at most of our modules to get an idea of the formatting we use.Personally, I tend to only use thirdparty modules with a decent unit test coverage, have you considered writing some for your more visible modules? It can give other devs more confidence that the module is working correctly, as well as help you to maintain such a massive list of modules through their lifecycle. I know there's commercial background behind most of these modules, but I'd argue that your clients would benefit from this as well
Oh, and there's a syntax error in http://sunny.svnrepository.com/sunny-side-up-general/trac.cgi/browser/vimeoembed/code/model/VimeoDataObject.php#L75 ;)
-
Re: a bunch of new modules

12 October 2010 at 2:04pm
Hi Ingo
Thank you for your encouraging comments.
I am totally keen for unit tests, however, despite two serious attempts, I am still unable to setup PHPUnit on my local machine. That is a real shame, because I am sure that otherwise all the modules would have some solid unit tests behind them. I might give that PHP Unit setup another go...
Thank you again for your positive feedback. I will fix that Vimeo error ;-)
Cheers
Nicolaas
-
Re: a bunch of new modules

12 October 2010 at 9:24pm
Oh, if thats all that stops you from writing tests, great
Feel free to hit me up on Skype about walking you through the PHPUnit setup, it should be really straightforward (http://www.phpunit.de/manual/current/en/installation.html) -
Re: a bunch of new modules

12 October 2010 at 10:13pm
These are my notes from 7 months ago, hope they help...
/mysite/_config.php
Director::set_environment_type("test");ensure you have the correct version of php first (with WAMP) that is 5.2.6 if you have 5.3.0 (default installation) then...
click sys tray icon for wamp
php
php version
get more
follow prompts to install 5.2.6open up the command line:
cd \wamp\bin\php\<phpversion>\
go-pear.bat
pear channel-discover pear.phpunit.de
pear install phpunit/PHPunitif you get an error that states the version of the pear installer needs to be upgraded then enter... (using the correct version of pear it asks for)
pear upgrade --force http://pear.php.net/get/PEAR-1.8.1.tar
change PHP.ini to find the includes required by PHPUnit:
;;;;;;;;;;;;;;;;;;;;;;;;;
; Paths and Directories ;
;;;;;;;;;;;;;;;;;;;;;;;;;; UNIX: "/path1:/path2"
;include_path = ".:/php/includes"
;
; Windows: "\path1;\path2"
include_path = ".;c:\php\includes;C:\wamp\bin\php\php5.2.6\PEAR\"This is how to run a unit test from the command line
php phpunit StackTest C:\wamp\www\unittesting\StackTest.php
Then the test is run with the webinterface by using :
http://localhost/mysite/dev/tests/xxxTest
-
Re: a bunch of new modules

12 October 2010 at 11:45pm
ok, great - thank you... I have done all this before.... but I tried it again...
I got as far as:
D:\wamp\bin\php\php5.2.6>pear install --force phpunit/PHPunit
Did not download optional dependencies: pear/Image_GraphViz, pear/Log, channel:/
/pear.symfony-project.com/YAML, use --alldeps to download automatically
warning: phpunit/PHPUnit requires PEAR Installer (version >= 1.8.1), installed v
ersion is 1.7.1
phpunit/PHPUnit can optionally use package "pear/Image_GraphViz" (version >= 1.2
.1)
phpunit/PHPUnit can optionally use package "pear/Log"
phpunit/PHPUnit can optionally use package "channel://pear.symfony-project.com/Y
AML" (version >= 1.0.2)
phpunit/PHPUnit can optionally use PHP extension "pdo_sqlite"
phpunit/PHPUnit can optionally use PHP extension "soap"
phpunit/PHPUnit can optionally use PHP extension "xdebug" (version >= 2.0.5)
downloading PHPUnit-3.4.15.tgz ...
Starting to download PHPUnit-3.4.15.tgz (255,036 bytes)
.....................................................done: 255,036 bytes
Validation Error: This package.xml requires PEAR version 1.8.1 to parse properly
, we are version 1.7.1
Parsing of package.xml from file "pear\tmp/package.xml" failed
Download of "phpunit/PHPunit" succeeded, but it is not a valid package archive
Error: cannot download "phpunit/PHPUnit"
Download failed
install failedI have upgrade to pear 1.9.1 , but that does not seem to get registered for some reason. No idea why.
Any hints?
Cheers
nicolaas
-
Re: a bunch of new modules

13 October 2010 at 12:13am
hmmmm, maybe it likes only 1.81 for phpunit?
do you have a "clean" machine that you can start again and ensure you force the right version with "pear upgrade --force http://pear.php.net/get/PEAR-1.8.1.tar"?
-
Re: a bunch of new modules

13 October 2010 at 12:20am
ok, when I type
pear -V
I get:
D:\wamp\bin\php\php5.2.6>pear upgrade --force http://pear.php.net/get/PEAR-1.8.1
.tar
downloading PEAR-1.8.1.tar ...
Starting to download PEAR-1.8.1.tar (Unknown size)
................................................................................
................................................................................
................................................................................
................................................................................
...............................done: 1,781,248 bytes
upgrade ok: channel://pear.php.net/PEAR-1.8.1
PEAR: Optional feature webinstaller available (PEAR's web-based installer)
PEAR: Optional feature gtkinstaller available (PEAR's PHP-GTK-based installer)
PEAR: Optional feature gtk2installer available (PEAR's PHP-GTK2-based installer)PEAR: To install optional features use "pear install pear/PEAR#featurename"
D:\wamp\bin\php\php5.2.6>pear -V
PEAR Version: 1.7.1
PHP Version: 5.2.6
Zend Engine Version: 2.2.0
Running on: Windows NT ASUS 6.1 build 7600D:\wamp\bin\php\php5.2.6>
Hmmm, somehow the upgrade does not seem to work.
Any hints?
| 2408 Views | ||
| Go to Top | Next > |


