Problem/Motivation

Nowadays many locations such as shops, companies even monuments has own webpage so in my opinion it should be possible by default to provide webadress for such location.

TODO:
Web adress validation.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

podarok’s picture

Status: Active » Needs work

You should provide a code in git diff format
http://drupal.org/patch
Goof to see here a tests (look at /tests folder for examples)

mist’s picture

Sorry for that but at this moment I am unfamiliar with git. If you could help me with this it would be great.

Yurii Krysiuk’s picture

Assigned: Unassigned » Yurii Krysiuk
Issue summary: View changes
Yurii Krysiuk’s picture

Status: Needs work » Needs review
FileSize
7.37 KB

Created patch file

moymilo’s picture

Assigned: Yurii Krysiuk » Unassigned
podarok’s picture

Status: Needs review » Needs work
  1. +++ b/location_www.info
    @@ -0,0 +1,16 @@
    +files[] = location_www.module
    

    There are no classes within a file. Let's do not overload autoloader

  2. +++ b/location_www.info
    @@ -0,0 +1,16 @@
    +files[] = location_www.install
    

    The same

  3. +++ b/location_www.info
    @@ -0,0 +1,16 @@
    +; Information added by drupal.org packaging script on 2012-07-25
    +version = "7.x-3.0-alpha1"
    +core = "7.x"
    +project = "location"
    +datestamp = "1343220794"
    

    No needs to add this, because it will be added by drupal.org automatically. Let's get rid of it

Yurii Krysiuk’s picture

Added new patch file

Yurii Krysiuk’s picture

Status: Needs work » Needs review
podarok’s picture

Status: Needs review » Fixed

Thanks, commited

  • podarok committed 97f3a31 on 7.x-3.x authored by mist
    Issue #1954248 by Yuriy Krysiuk, mist: Location WWW contrib module
    

Status: Fixed » Needs work

The last submitted patch, 7: Location_www_1954248_5.patch, failed testing.