Skip to content

Commit

Permalink
Improved AcceptedResponse and RQL message errors
Browse files Browse the repository at this point in the history
  • Loading branch information
danieleteti committed Feb 4, 2025
1 parent 597faaa commit dbe00bf
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
4 changes: 4 additions & 0 deletions sources/MVCFramework.RQL.Parser.pas
Original file line number Diff line number Diff line change
Expand Up @@ -622,6 +622,10 @@ procedure TRQL2SQL.ParseBinOperator(const aToken: TRQLToken; const aAST: TObject
Error('Unclosed string');
lValueType := vtString;
end
else if lToken = tkQuote then {special error case}
begin
Error('String literals must be enclosed in Double Quotes - HINT: Convert single quotes into double quotes ( '' -> " )');
end
else if (aToken in [tkIn, tkOut]) and (lToken = tkOpenBracket) then
begin
lList := TList<string>.Create;
Expand Down
8 changes: 6 additions & 2 deletions sources/MVCFramework.pas
Original file line number Diff line number Diff line change
Expand Up @@ -786,7 +786,7 @@ TMVCRenderer = class(TMVCBase)
function CreatedResponse(const Location: string = ''; const Body: TObject = nil): IMVCResponse; overload;
function CreatedResponse(const Location: string; const Message: String): IMVCResponse; overload;

function AcceptedResponse(const Location: string = ''; const Body: TObject = nil): IMVCResponse;
function AcceptedResponse(const Location: string = ''; const Body: TObject = nil; const Message: String = ''): IMVCResponse; overload;

function ConflictResponse: IMVCResponse;

Expand Down Expand Up @@ -3943,7 +3943,7 @@ function TMVCController.GetClientId: string;
end;

function TMVCRenderer.AcceptedResponse(const Location: string;
const Body: TObject): IMVCResponse;
const Body: TObject; const Message: String): IMVCResponse;
var
lRespBuilder: IMVCResponseBuilder;
begin
Expand All @@ -3956,6 +3956,10 @@ function TMVCRenderer.AcceptedResponse(const Location: string;
begin
lRespBuilder.Body(Body, True);
end;
if not Message.IsEmpty then
begin
lRespBuilder.Body(Message);
end;
Result := lRespBuilder.StatusCode(HTTP_STATUS.Accepted).Build;
end;

Expand Down

0 comments on commit dbe00bf

Please sign in to comment.