OpencachingDeutschland/oc-server3
1
0
Fork 0
Code Issues Pull Requests Releases Activity

fixed several adoption problems, fixes #216

Browse Source 
- adoption did not work at all since commit cd53979 (because of action="... fix)
- redirect to wrong page after withdrawing adoption offer
- cryptic error message "unknown error" when offering to a disabled user
- messages semantic errors and formatting
This commit is contained in:
following
2013-06-05 13:37:23 +02:00
parent 91d98f7e81
commit 07a1ae0185
4 changed files with 42 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
htdocs/adoptcache.php
View File

@ -17,6 +17,8 @@
$action = isset($_REQUEST['action']) ? $_REQUEST['action'] : 'listbyuser';
$tpl->assign('action', $action);
$tpl->assign('error', '');
if (isset($_REQUEST['cacheid']))
$tpl->assign('cacheid', $_REQUEST['cacheid']+0);
if ($action == 'listbycache')
{
@ -150,20 +152,13 @@ function addRequest($cacheid, $userid)
if ($cache->exist() == false)
$tpl->error(ERROR_CACHE_NOT_EXISTS);
if ($cache->allowEdit() == false)
$tpl->error(ERROR_NO_ACCESS);
if ($cache->getUserId() == $userid)
{
$tpl->assign('error', 'sameuser');
$tpl->display();
}
$adopt_result = $cache->addAdoption($userid);
if ($adopt_result === true)
$tpl->redirect('adoptcache.php?action=listbycache&cacheid=' . $cacheid);
else
{
if ($cache->addAdoption($userid) == false)
$tpl->error(ERROR_UNKNOWN);
$tpl->redirect('adoptcache.php?action=listbycache&cacheid=' . $cacheid);
$tpl->assign('error',$adopt_result);
listRequestsByCacheId($cacheid);
}
}
@ -197,6 +192,10 @@ function cancelRequest($cacheid, $userid)
if ($cache->cancelAdoption($userid) == false)
$tpl->error(ERROR_UNKNOWN);
$tpl->redirect('adoptcache.php');
if ($userid == $login->userid)
$tpl->redirect('adoptcache.php');
else
$tpl->redirect('adoptcache.php?action=listbycache&cacheid=' . $cacheid);
}
?>

23
htdocs/doc/sql/static-data/data.sql
View File

@ -1685,8 +1685,8 @@ INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('827', 'Adoption
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('828', 'Date', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('829', 'Yes, i\'ve read and understand the <a href=\"articles.php?page=impressum#tos\">terms of use</a> of Opencaching.de', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('830', 'To complete the adoption, you have to submit this form.<br /> With submitting this form you are the responsible owner of this Geocache.', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('831', '%1 offered you this Geocache for adoption at %2', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('832', 'Thank you for adopting %1', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('831', '<strong>%1</strong> offered you this Geocache for adoption at %2.', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('832', 'Thank you for adopting <strong>%1</strong>.', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('833', 'Commit the adoption of %1', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('834', 'There are no caches that can be adopted by you at the moment.', '2010-08-28 11:48:04');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('835', 'Adopt now', '2010-08-28 11:48:04');
@ -2491,6 +2491,7 @@ INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('2069', 'since F
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('2070', 'One ore more emails to this user could not be delivered. It might be a good idea to additionally log comments on the user\'s geocaches, and/or trying to contact him/her through other channels like a message board account or another geocaching platform.', '2013-05-28 16:51:40');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('2071', 'This cache has been \"temporarily unavailable\" for more than one year now; therefore it is being archived automatically. The owner may decide to maintain the cache and re-enable the listing.', '2013-05-28 16:51:40');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('2072', 'This event took place more than five weeks ago; therefore it is being archived automatically. The owner may re-enable the listing if it should stay active for some exceptional reason.', '2013-05-28 16:51:40');
INSERT INTO `sys_trans` (`id`, `text`, `last_modified`) VALUES ('2073', 'This user account is disabled.', '2013-05-28 16:51:40');
-- Table sys_trans_ref
SET NAMES 'utf8';
@ -5630,8 +5631,8 @@ INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUE
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('828', 'DE', 'Datum', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('829', 'DE', 'Ja, ich habe die <a href=\"articles.php?page=impressum#tos\">Nutzungsbedingungen</a> von Opencaching.de gelesen und werde sie befolgen.', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('830', 'DE', 'Um die Adoption abzuschließen, musst du diese Seite bestätigen.<br />\r\nNachdem du die Seite bestätigt hast, bist du für diesen Geocache als Besitzer verantwortlich.', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'DE', '%1 hat dir diesen Geocache am %2 zur Adoption angeboten', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'DE', 'Vielen Dank für die Adoption von %1', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'DE', '<strong>%1</strong> hat dir diesen Geocache am %2 zur Adoption angeboten.', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'DE', 'Vielen Dank für die Adoption von <strong>%1</strong>.', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('833', 'DE', 'Die Adoption von %1 bestätigen', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('834', 'DE', 'Es werden dir derzeit keine Geocaches zur Adoption angeboten.', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('835', 'DE', 'Adoptieren', '2010-08-28 11:48:07');
@ -6434,6 +6435,7 @@ INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUE
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2070', 'DE', 'Eine oder mehrere Emails an diesen Benutzer konnten nicht zugestellt werden. Es könnte ratsam sein, Hinweise zu seinen Caches zusätzlich zu loggen, und ihn ggf. über weitere Adressen wie z.B. Forenaccounts oder andere Geocaching-Plattformen zu kontaktieren.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2071', 'DE', 'Dieser Cache ist seit mehr als einem Jahr &bdquo;momentan nicht verfügbar&ldquo;; daher wird er automatisch archiviert. Er kann vom Besitzer jederzeit instand gesetzt und reaktiviert werden.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2072', 'DE', 'Dieses Event fand vor über fünf Wochen statt; daher wird es automatisch archiviert. Das Listing kann vom Owner selbst reaktiviert werden, falls es aus besonderen Gründen weiter aktiv bleiben soll.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2073', 'DE', 'Dieser Benutzer ist deaktiviert.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('1', 'EN', 'Reorder IDs \r', '2010-09-02 00:15:30');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2', 'EN', 'The database could not be reconnected.', '2010-08-28 11:48:07');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('3', 'EN', 'Testing – please do not login', '2010-08-28 11:48:07');
@ -7226,8 +7228,8 @@ INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUE
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('828', 'EN', 'Date', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('829', 'EN', 'Yes, I have read and will comply with the Opencaching.de <a href=\"articles.php?page=impressum#tos\">terms of use</a>.', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('830', 'EN', 'To complete the adoption, you have to submit this form.<br /> By submitting this form, you will become the responsible owner of this geocache.', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'EN', 'on %2, %1 offered this geocache to you for adoption ', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'EN', 'Thank you for adopting %1', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'EN', 'on %2, <strong>%1</strong> offered this geocache to you for adoption.', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'EN', 'Thank you for adopting <strong>%1</strong>.', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('833', 'EN', 'Commit the adoption of %1', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('834', 'EN', 'There are no caches that can be adopted by you at the moment.', '2010-08-28 11:48:08');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('835', 'EN', 'Adopt now', '2010-08-28 11:48:08');
@ -7997,6 +7999,7 @@ INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUE
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2070', 'EN', 'One ore more emails to this user could not be delivered. It might be a good idea to additionally log comments on the user\'s geocaches, and/or trying to contact him/her through other channels like a message board account or another geocaching platform.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2071', 'EN', 'This cache has been \"temporarily unavailable\" for more than one year now; therefore it is being archived automatically. The owner may decide to maintain the cache and re-enable the listing.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2072', 'EN', 'This event took place more than five weeks ago; therefore it is being archived automatically. The owner may re-enable the listing if it should stay active for some exceptional reason.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2073', 'EN', 'This user account is disabled.', '2013-04-25 23:00:00');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('1', 'ES', 'Reordenar ID', '2010-12-09 00:17:55');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('2', 'ES', 'La base de datos no se pudo conectar.', '2010-12-09 00:17:55');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('3', 'ES', 'En pruebas - por favor, no entre.', '2010-12-09 00:17:55');
@ -8789,8 +8792,8 @@ INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUE
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('828', 'ES', 'Data', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('829', 'ES', 'Sí, lo he leído y aprobado las <a href=\"articles.php?page=impressum#tos\">condiciones de uso</a> de Opencaching.de.', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('830', 'ES', 'Para completar la adopción, debe enviar este form.<br /> Al enviar este formulario, usted se convierte en el propietario responsable de esta geocache.', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'ES', 'el %2, %1 me dio en adopción este geocache', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'ES', 'Gracias por haber adoptado %1', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'ES', 'el %2, <strong>%1</strong> me dio en adopción este geocache.', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'ES', 'Gracias por haber adoptado <strong>%1</strong>.', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('833', 'ES', 'Confirma la adopción de %1', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('834', 'ES', 'En este momento hay caches que pueden ser adoptados.', '2010-12-09 00:17:59');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('835', 'ES', 'Adoptar ahora', '2010-12-09 00:17:59');
@ -10342,8 +10345,8 @@ INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUE
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('828', 'IT', 'Data', '2010-08-31 20:06:24');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('829', 'IT', 'Si, ho letto e approvato le <a href=\"articles.php?page=impressum#tos\">condizioni d\'uso</a> di Opencaching.de.', '2010-09-01 23:49:32');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('830', 'IT', 'Per completare l\'adozione, devi inviare questa form.<br /> Inviando questa form, diventi il proprietario responsabile per questa geocache.', '2010-08-31 20:06:24');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'IT', 'il %2, %1 ti ha offerto questa geocache in adozione', '2010-09-01 23:49:32');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'IT', 'Grazie per avere adottato %1', '2010-10-27 18:49:19');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('831', 'IT', 'il %2, <strong>%1</strong> ti ha offerto questa geocache in adozione.', '2010-09-01 23:49:32');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('832', 'IT', 'Grazie per avere adottato <strong>%1</strong>.', '2010-10-27 18:49:19');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('833', 'IT', 'Conferma l\'adozione di %1', '2010-09-01 23:49:32');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('834', 'IT', 'Non ci sono al momento attuale cache che possano essere adottate.', '2010-08-31 20:06:24');
INSERT INTO `sys_trans_text` (`trans_id`, `lang`, `text`, `last_modified`) VALUES ('835', 'IT', 'Adotta adesso', '2010-09-01 23:49:32');

9
htdocs/lib2/logic/cache.class.php
View File

@ -368,14 +368,17 @@ class cache
function addAdoption($userid)
{
if ($this->allowEdit() == false)
return false;
return "noaccess";
if (sql_value("SELECT COUNT(*) FROM `user` WHERE `user_id`='&1'", 0, $userid) == 0)
return "userunknown";
if (sql_value("SELECT COUNT(*) FROM `user` WHERE `user_id`='&1' AND `is_active_flag`=1", 0, $userid) == 0)
return false;
return "userdisabled";
// same user?
if ($this->getUserId() == $userid)
return false;
return "self";
sql("INSERT IGNORE INTO `cache_adoption` (`cache_id`, `user_id`) VALUES ('&1', '&2')", $this->nCacheId, $userid);

17
htdocs/templates2/ocstyle/adoptcache.tpl
View File

@ -8,6 +8,7 @@
<form method="post" action="adoptcache.php">
<input type="hidden" name="action" value="add" />
<input type="hidden" name="submit" value="1" />
<input type="hidden" name="cacheid" value="{$cacheid}" />
<div class="content2-pagetitle">
<img src="resource2/{$opt.template.style}/images/profile/22x22-email.png" style="align: left; margin-right: 10px;" width="22" height="22" alt="" />
@ -61,12 +62,14 @@
</tr>
<tr><td colspan="3">&nbsp;</td></tr>
<tr>
<td>{t}Username:{/t}</td>
<td width="22%">{t}Username:{/t}</td>
<td colspan="2"><input type="text" name="username" value="{$adoptusername|escape}" size="40" /></td>
</tr>
{if $error=='userunknown'}
<tr><td>&nbsp;</td><td colspan="2"><span class="errormsg">{t}Username unknown{/t}</span></td></tr>
{elseif $error=='sameuser'}
{elseif $error=='userdisabled'}
<tr><td>&nbsp;</td><td colspan="2"><span class="errormsg">{t}This user account is disabled.{/t}</span></td></tr>
{elseif $error=='self'}
<tr><td>&nbsp;</td><td colspan="2"><span class="errormsg">{t}You cannot adopt your own cache!{/t}</span></td></tr>
{/if}
<tr>
@ -126,15 +129,16 @@
</table>
{elseif $action=='commit'}
<form method="post" action"adoptcache.php">
<form method="post" action="adoptcache.php">
<input type="hidden" name="action" value="commit" />
<input type="hidden" name="submit" value="1" />
<input type="hidden" name="cacheid" value="{$cacheid}" />
<div class="content2-pagetitle"><img src="resource2/{$opt.template.style}/images/profile/22x22-email.png" style="align: left; margin-right: 10px;" width="22" height="22" alt="" />{t 1=$cache.name|escape}Commit the adoption of %1{/t}</div>
<p style="line-height: 1.6em;">
{t 1=$cache.name|escape}Thank you for adopting %1{/t}<br />
{t 1=$cache.username|escape 2=$cache.date_created|date_format:$opt.format.date}%1 offered you this Geocache for adoption at %2{/t}
{t 1=$cache.name|escape}Thank you for adopting <strong>%1</strong>.{/t}<br />
{t 1=$cache.username|escape 2=$cache.date_created|date_format:$opt.format.date}<strong>%1</strong> offered you this Geocache for adoption at %2.{/t}
</p>
<p style="line-height: 1.6em;">
@ -143,6 +147,7 @@
</p>
<p style="line-height: 1.6em;">
<br />
<input type="checkbox" name="tou" value="1" />
<label for="tou">{t}Yes, i've read and understand the <a href="articles.php?page=impressum#tos">terms of use</a> of Opencaching.de{/t}</label>
</p>
@ -152,6 +157,6 @@
{t}Sorry, you have to read and accept the terms of use to proceed!{/t}
</p>
{/if}
<p style="line-height: 1.6em;"><input type="submit" value="{t}Submit{/t}" /></p>
<p style="line-height: 1.6em;"><input type="submit" value="{t}Submit{/t}" class="formbutton" onclick="submitbutton('submit')" /></p>
</form>
{/if}