Looks like $items is undefined around line ~365 in drealty.daemon.php. When the importer jumps to the next property class, it just kept going in an endless loop for me.

The attached patch fixes the issue. Simply looks like $count was missing, and $items needed replacing.

CommentFileSizeAuthor
#1 1512780.patch1.43 KBkevinquillen

Comments

kevinquillen’s picture

kevinquillen commented
Version: 7.x-3.0-beta1 » 7.x-3.x-dev
Status: Active » Needs review
StatusFileSize
new 1512780.patch1.43 KB

Attaching patch. This jumpstarted my import again, and switching property classes does not result in drush stuck in a loop.

kevinquillen’s picture

kevinquillen commented

Now I sometimes get this drush error:

id string:                                                                                                                                           [notice]
594129,594133,593163,566929,592747,592238,592239,592241,592742,592626,560994,591230,570424,589768,589599,589597,580852,580866,580878,590794
[137.38 sec, 142.65 MB]
Undefined offset:  -9223372036854775808 drealty.daemon.php:691 [137.5 sec, 142.65 MB]                                                                [notice]
Downloaded a total of 0 images for 20 Listings. [NAN of data]. [137.5 sec, 142.65 MB]                                                                [notice]
Disconnected from RETS server. [137.54 sec, 142.64 MB]

Resulting in no images downloaded.

drealty.daemon.php line 691:

public function formatBytes($size, $precision = 2) {
    $base = log($size) / log(1024);
    $suffixes = array('', 'k', 'M', 'G', 'T');
    return round(pow(1024, $base - floor($base)), $precision) . $suffixes[floor($base)];
  }
kevinquillen’s picture

kevinquillen commented
Title: Undefined index $items in drealty.daemon.php » Undefined offset error in drealty.daemon.php
camidoo’s picture

camidoo commented

We can actually remove that whole call, as it's just a 'pretty' thing to see in the log and has no real bearing on the import.

camidoo’s picture

camidoo commented
Status: Needs review » Fixed

I removed it, shouldn't be causing anymore problems.

kevinquillen’s picture

kevinquillen commented

Well, it did say this multiple times:

Downloaded a total of 0 images for 20 Listings. [NAN of data]. [137.5 sec, 142.65 MB]   
Downloaded a total of 0 images for 20 Listings. [NAN of data]. [137.5 sec, 142.65 MB]   
Downloaded a total of 0 images for 20 Listings. [NAN of data]. [137.5 sec, 142.65 MB]   
Downloaded a total of 0 images for 20 Listings. [NAN of data]. [137.5 sec, 142.65 MB]   
Downloaded a total of 0 images for 20 Listings. [NAN of data]. [137.5 sec, 142.65 MB]

Was that due to an actual error or just logging error?

camidoo’s picture

camidoo commented

That'd just be a logging error

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.