diff options
author | friendica <info@friendica.com> | 2014-06-30 21:17:19 -0700 |
---|---|---|
committer | friendica <info@friendica.com> | 2014-06-30 21:17:19 -0700 |
commit | b2abc519296570b0dd362cc3a0e1c6f1da3b989f (patch) | |
tree | 9e56b17649db325e4ac6dd4e941b0c014c02b373 | |
parent | 4fbe63aaeba63ed416e4990416bfcd5a8f5f3e33 (diff) | |
download | volse-hubzilla-b2abc519296570b0dd362cc3a0e1c6f1da3b989f.tar.gz volse-hubzilla-b2abc519296570b0dd362cc3a0e1c6f1da3b989f.tar.bz2 volse-hubzilla-b2abc519296570b0dd362cc3a0e1c6f1da3b989f.zip |
much better fix to problem yesterday of updates with the wrong ud_addr getting updated. Now we'll
pass in the update table row to import_xchan so we know exactly which ud_addr applies. We still need ud_flags passed in separately.
-rw-r--r-- | include/dir_fns.php | 2 | ||||
-rw-r--r-- | include/zot.php | 39 |
2 files changed, 17 insertions, 24 deletions
diff --git a/include/dir_fns.php b/include/dir_fns.php index 2f4830d05..1bc97cfec 100644 --- a/include/dir_fns.php +++ b/include/dir_fns.php @@ -157,7 +157,7 @@ function update_directory_entry($ud) { $j = json_decode($x['body'],true); if($j) $success = true; - $y = import_xchan($j,0); + $y = import_xchan($j,0,$ud); } if(! $success) { $r = q("update updates set ud_last = '%s' where ud_addr = '%s'", diff --git a/include/zot.php b/include/zot.php index 8401ac186..798ebc0af 100644 --- a/include/zot.php +++ b/include/zot.php @@ -582,16 +582,19 @@ function zot_register_hub($arr) { * * @param array $arr => json_decoded discovery packet * @param int $ud_flags - * Determines whether to create a directory update record if any changes occur, default is UPDATE_FLAGS_UPDATED (true) + * Determines whether to create a directory update record if any changes occur, default is UPDATE_FLAGS_UPDATED * $ud_flags = UPDATE_FLAGS_FORCED indicates a forced refresh where we unconditionally create a directory update record * this typically occurs once a month for each channel as part of a scheduled ping to notify the directory * that the channel still exists + * @param array $ud_arr + * If set [typically by update_directory_entry()] indicates a specific update table row and more particularly + * contains a particular address (ud_addr) which needs to be updated in that table. * * @returns array => 'success' (boolean true or false) * 'message' (optional error string only if success is false) */ -function import_xchan($arr,$ud_flags = UPDATE_FLAGS_UPDATED) { +function import_xchan($arr,$ud_flags = UPDATE_FLAGS_UPDATED, $ud_arr = null) { call_hooks('import_xchan', $arr); @@ -601,7 +604,6 @@ function import_xchan($arr,$ud_flags = UPDATE_FLAGS_UPDATED) { $changed = false; $what = ''; - $addresses = array(); if(! (is_array($arr) && array_key_exists('success',$arr) && $arr['success'])) { logger('import_xchan: invalid data packet: ' . print_r($arr,true)); @@ -838,8 +840,6 @@ function import_xchan($arr,$ud_flags = UPDATE_FLAGS_UPDATED) { if(strpos($location['address'],'/') !== false) $location['address'] = substr($location['address'],0,strpos($location['address'],'/')); - $addresses[] = $location['address']; - // match as many fields as possible in case anything at all changed. $r = q("select * from hubloc where hubloc_hash = '%s' and hubloc_guid = '%s' and hubloc_guid_sig = '%s' and hubloc_url = '%s' and hubloc_url_sig = '%s' and hubloc_host = '%s' and hubloc_addr = '%s' and hubloc_callback = '%s' and hubloc_sitekey = '%s' ", @@ -964,15 +964,16 @@ function import_xchan($arr,$ud_flags = UPDATE_FLAGS_UPDATED) { } + // Which entries in the update table are we interested in updating? - + $address = (($ud_arr && $ud_arr['ud_addr']) ? $ud_arr['ud_addr'] : $arr['address']); // Are we a directory server of some kind? if($dirmode != DIRECTORY_MODE_NORMAL) { if(array_key_exists('profile',$arr) && is_array($arr['profile'])) { - $profile_changed = import_directory_profile($xchan_hash,$arr['profile'],$arr['address'],$ud_flags, 1); + $profile_changed = import_directory_profile($xchan_hash,$arr['profile'],$address,$ud_flags, 1); if($profile_changed) { $what .= 'profile '; $changed = true; @@ -998,27 +999,19 @@ function import_xchan($arr,$ud_flags = UPDATE_FLAGS_UPDATED) { } } + if(($changed) || ($ud_flags == UPDATE_FLAGS_FORCED)) { - if($addresses) { - foreach($addresses as $address) { - $guid = random_string() . '@' . get_app()->get_hostname(); - update_modtime($xchan_hash,$guid,$address,$ud_flags); - } - } + $guid = random_string() . '@' . get_app()->get_hostname(); + update_modtime($xchan_hash,$guid,$address,$ud_flags); logger('import_xchan: changed: ' . $what,LOGGER_DEBUG); } elseif(! $ud_flags) { // nothing changed but we still need to update the updates record - if($addresses) { - foreach($addresses as $address) { - - q("update updates set ud_flags = ( ud_flags | %d ) where ud_addr = '%s' and not (ud_flags & %d) ", - intval(UPDATE_FLAGS_UPDATED), - dbesc($address), - intval(UPDATE_FLAGS_UPDATED) - ); - } - } + q("update updates set ud_flags = ( ud_flags | %d ) where ud_addr = '%s' and not (ud_flags & %d) ", + intval(UPDATE_FLAGS_UPDATED), + dbesc($address), + intval(UPDATE_FLAGS_UPDATED) + ); } if(! x($ret,'message')) { |