search

UofS-Pulse-Binfo / nd_genotypes

Drupal/Tripal: Tripal interface for Chado Natural Diversity Genotypic data.
https://nd-genotypes.readthedocs.io/en/latest/index.html
GNU General Public License v2.0
4 stars 1 forks source link

PHP Fatal error: Cannot redeclare nd_genotypes_genotype_call_schema_template() #54

Closed Xaiku closed 3 years ago

Xaiku commented 3 years ago

Hi everybody,

I uninstalled and reinstalled nd_genotypes and this is the error I got after trying to pm-enable nd_genotypes: image

I looked into the files and here are some screenshots: genotypes_loader.install image nd_genotypes.mview.inc image

The two files contain the same function. Do I have to manually delete 1 function from 1 file or is there another way?

laceysanderson commented 3 years ago

Oh no 😶 This was missed when we upgraded the genotypes loader to be independent from nd_genotypes and is a copy/paste error. I have a PR to fix it on Genotypes Loader which will be merged as soon as tests pass and will resolve this.

laceysanderson commented 3 years ago

There. It has been merged into master for Genotypes Loader. Please update your version of that module and then try again. So Sorry for the difficulties!!!

Xaiku commented 3 years ago

Hey @laceysanderson,

I updated my genotypes_loader version, but when I try to pm-enable genotypes_loader it gives me the following error: image

laceysanderson commented 3 years ago

I'm so sorry -this is what I get for fixing bugs on the weekend :-(

I have removed the duplicate function in 7336dcc and tested importing genotypes manually according to https://genotypes-loader.readthedocs.io/en/latest/demo.html with nd_genotypes already installed.

Xaiku commented 3 years ago

I updated the genotypes_loader folder and I got no errors with enabling genotypes_loader and nd_genotypes with pm-enable. Thank you for your help!