Opened 16 years ago

Closed 16 years ago

#2142 closed New Feature (fixed)

HTML samples should use php extension in action paramter inside a form

Reported by: Wojciech Olchawa Owned by: Wojciech Olchawa
Priority: Normal Milestone: FCKeditor 2.6.1
Component: General Version: SVN (FCKeditor) - Retired
Keywords: Confirmed Review+ Cc:

Description

My suggestion is to replace sampleposteddata.asp with sampleposteddata.php in action parameter inside the form. I think that the usage of asp extension is less useful for the users. I don't know what is the most common used server side however I think that PHP is more popular than ASP (more of the users use Appache rather than IIS for example). Of course a user could always use the php examples however I don't see why php extension couldn't be used in html samples as well.

Attachments (2)

2142.patch (12.2 KB) - added by Wojciech Olchawa 16 years ago.
2142_2.patch (12.6 KB) - added by Wojciech Olchawa 16 years ago.
Patch proposal

Download all attachments as: .zip

Change History (14)

comment:1 Changed 16 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added; Discussion removed
Milestone: FCKeditor 2.6.1

It makes a lot of sense. PHP is definitely the most used language, related to the editor (we have done a research about it previously).

comment:2 Changed 16 years ago by Wojciech Olchawa

Owner: set to Wojciech Olchawa

comment:3 Changed 16 years ago by Wojciech Olchawa

I've attached a patch proposal. The patch should throw an error when trying to modify sampleposteddata.asp from _samples/html catalog. (because of attempt to delete it). However it shouldn't be a problem when committing the patch.

Changed 16 years ago by Wojciech Olchawa

Attachment: 2142.patch added

comment:4 Changed 16 years ago by Wojciech Olchawa

Keywords: Review? added

comment:5 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review+ added; Review? removed

Please, update the what's new file remarking that it's an important change from previous versions.

comment:6 Changed 16 years ago by Wojciech Olchawa

Resolution: fixed
Status: newclosed

Fixes with [1947]

comment:7 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: closedreopened

Now I think that instead of copying the sampleposteddata.php it would be better to change the action so it points to ../php/sampleposteddata.php

That way there's only one copy of that file and it's absolutely clear for anyone what to do if they are working with other languages.

comment:8 Changed 16 years ago by Wojciech Olchawa

Good point I will provide a new patch in a few minutes

comment:9 Changed 16 years ago by Wojciech Olchawa

Keywords: Review+ removed

comment:10 Changed 16 years ago by Wojciech Olchawa

Keywords: Review? added

I don't see any reason for keeping sampleposteddata.html as well. I have applied changes to the patch.

Changed 16 years ago by Wojciech Olchawa

Attachment: 2142_2.patch added

Patch proposal

comment:11 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review+ added; Review? removed

comment:12 Changed 16 years ago by Wojciech Olchawa

Resolution: fixed
Status: reopenedclosed

Fixed with [1957]

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy