1)Geofield fields contain nine columns of information about the geographic data that is stores.
that is stores => that it stores
2) overriden>>>overridden
3) the field according ton the chosen options. Change "ton" to "to"

CommentFileSizeAuthor
#12 geofield_fix-2863281-12.patch3.6 KBdbjpanda
#9 geofield_fix-2863281-9.patch3.48 KBdhruveshdtripathi
#5 typo_fixed_2863281_5.patch1.58 KBdbjpanda
#2 Typo_fixed.patch578 bytesdbjpanda
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dbjpanda created an issue. See original summary.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
FileSize
Typo_fixed.patch578 bytes

Corrected the typo.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
Status: Active » Needs review
riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Needs work

@dbjpanda, Thanks for corrections but there is 2 more typos found from the file which i mentioned below.

1) overriden>>>overridden
2) the field according ton the chosen options. Change "ton" to "to"

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
FileSize
typo_fixed_2863281_5.patch1.58 KB

@ Jigar.addweb Thanks for pointing out. corrected as per your concern.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
Title: Fix a typo » geofield: fix readme.txt documentation typo
Issue summary: View changes
Status: Needs work » Needs review
riddhi.addweb’s picture

riddhi.addweb CreditAttribution: riddhi.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Needs work

@dbjpanda, Thanks for the quick response but i found some more issues in ReadMe file which i added below :

1) Use Breadcrumbs instead of the path i.e. "/admin/structure/openlayers/maps." Need for a change it throughout the file.
2) Path for installing module link from drupal.org is break.

INSTALL
-------

Install the modules Geofield and geoPHP in the usual way. General information
on installing Drupal modules can be found here: http://drupal.
org/documentation/install/modules-themes/modules-7
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Assigned: Unassigned » dhruveshdtripathi

Working on it

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
FileSize
geofield_fix-2863281-9.patch3.48 KB

Made changes described in comment #7. Also made some changes according to README text standards.

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Assigned: dhruveshdtripathi » Unassigned
Status: Needs work » Needs review
dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
Status: Needs review » Needs work

Thanks. But the url should not be alias of any node. We should always include the permanent url as per https://www.drupal.org/docs/7/extending-drupal-7/installing-contributed-... instruction.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
FileSize
geofield_fix-2863281-12.patch3.6 KB

Please review. Slight changes done.

dbjpanda’s picture

dbjpanda CreditAttribution: dbjpanda commented
Status: Needs work » Needs review
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Status: Needs review » Reviewed & tested by the community

Agreed @dbjpanda. Changes are good.