I don't relish the idea of the importer code being imported and parsed on every page load. Please split it into a separate module since most people who use this are not coming from pligg, and nobody needs to run the code more than once. Thanks.

Comments

mercmobily’s picture

mercmobily commented

Hi,

I totally agree.
Rather than writing another module, I thought about a checkbox which, if clicked, loads the extra code...
Maybe that's silly?

But again, I agree. It's tons of (ugly) code. It's ugly because it has a limited lifespan, and I didn't want to spend too long writing it. It's also DANGEROUS code - it will WIPE your existing Drupal installation effectively... and replace it with the data imported from Pligg.

But, in general, I totally agree. Will do.

Merc.

robertdouglass’s picture

robertdouglass commented

I think a separate module is the best option, much better than a config option.

mercmobily’s picture

mercmobily commented

Hi,

OK, will do it that way.

Merc.

mercmobily’s picture

mercmobily commented
Status: Active » Closed (fixed)

Hi,

OK, done.

Merc.