oqapi/webgui
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

IE8 on XP demands that onclick actions return false, otherwise it submits the form twice. Fixes bug #11336.

Browse source
This commit is contained in:
Colin Kuskie 2010-02-09 12:06:25 -08:00
parent d31be40a59
commit fb3211aa3c
3 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

1
docs/changelog/7.x.x.txt
View file

@ -6,6 +6,7 @@
- fixed #11395: i18n LDAP field name in user properties wrong - fixed #11395: i18n LDAP field name in user properties wrong
- fixed #11396: WebGUI Workflow Cache Error. - fixed #11396: WebGUI Workflow Cache Error.
- fixed #11393: Map template variables missing - fixed #11393: Map template variables missing
- fixed #11336: Duplicate Version Tag Created.
7.8.11 7.8.11
- fixed #11362: Unable to checkout with ITransact plugin - fixed #11362: Unable to checkout with ITransact plugin

10
lib/WebGUI/Content/AssetManager.pm
View file

@ -411,11 +411,11 @@ ENDHTML
<div id="dataTableContainer"> <div id="dataTableContainer">
</div> </div>
<p class="actions"> %s <p class="actions"> %s
<input type="submit" name="action_update" value="%s" onclick="this.form.func.value='setRanks'; this.form.submit();" /> <input type="submit" name="action_update" value="%s" onclick="this.form.func.value='setRanks'; this.form.submit(); return false;" />
<input type="submit" name="action_delete" value="%s" onclick="if( confirm('%s')){ this.form.func.value='deleteList'; this.form.submit(); }{ return false; }" /> <input type="submit" name="action_delete" value="%s" onclick="if( confirm('%s')){ this.form.func.value='deleteList'; this.form.submit(); return false;}{ return false; }" />
<input type="submit" name="action_cut" value="%s" onclick="this.form.func.value='cutList'; this.form.submit();"/> <input type="submit" name="action_cut" value="%s" onclick="this.form.func.value='cutList'; this.form.submit(); return false;"/>
<input type="submit" name="action_copy" value="%s" onclick="this.form.func.value='copyList'; this.form.submit();"/> <input type="submit" name="action_copy" value="%s" onclick="this.form.func.value='copyList'; this.form.submit(); return false;"/>
<input type="submit" name="action_duplicate" value="%s" onclick="this.form.func.value='duplicateList'; this.form.submit();"/> <input type="submit" name="action_duplicate" value="%s" onclick="this.form.func.value='duplicateList'; this.form.submit(); return false;"/>
</p> </p>
</form> </form>
<div id="pagination"> <div id="pagination">

2
lib/WebGUI/Session/Icon.pm
View file

@ -166,7 +166,7 @@ sub delete {
my $pageURL = shift || $self->session->url->getRequestedUrl; my $pageURL = shift || $self->session->url->getRequestedUrl;
my $confirmText = shift; my $confirmText = shift;
if($confirmText) { if($confirmText) {
$confirmText = qq| onclick="return confirm('$confirmText')" |; $confirmText = qq| onclick="return confirm('$confirmText');" |;
} }
my $i18n = WebGUI::International->new($self->session,'Icon'); my $i18n = WebGUI::International->new($self->session,'Icon');
my $output = '<p class="toolbarIcon" style="display:inline;vertical-align:middle;"><a href="'.$self->session->url->gateway($pageURL,$urlParams).'" '.$confirmText.'>'; my $output = '<p class="toolbarIcon" style="display:inline;vertical-align:middle;"><a href="'.$self->session->url->gateway($pageURL,$urlParams).'" '.$confirmText.'>';