macx / google-plus-photo-album Goto Github PK

Hey there! :)

Just overlooked your code and was wondering, if would be cooler, to have a factory, that returns a photo album for a specific provider. So the syntax could look like this:

$album = AlbumFactory::getAlbum(array(
  "id" => ID,
  "user_id" => ID
));

Then you could have a much more appropriate API:

$album->getImage(id);
$album->getImages();
$album->getTitle();
...
..

What do you think? :)

The namespace is currently defined as macx/.... I'm not entirely sure if this is a good pattern. Because the actual namespace of this class is... well the GooglePlusPhotoAlbum namespace (or whatever you want to call it). So what do you think about considering to change it?

leaving this here to keep track on the state of this issue. Also referencing #2

Write a easy to understand documentation of how to use the class. It needs two ways to implement it: Using package-manager composer or for the copy-and-paste users.