I am using the FileStack API and the filepicker gem (https://github.com/Ink/filepicker-rails). I follow the steps outlined in the doc under Accessing FilePicker File with OnChange. I am attempting to grab the JSON browser response attribute, filename, pass it to my Rails controller via a route, and save it as a :name in my Attachment model.
Attachment/New view
<%= filepicker_js_include_tag %>
<%= simple_form_for(#attachment) do |f| %>
<%= f.filepicker_field :title, multiple: 'true', onchange: 'onUpload(event)' %>
<%= f.submit %>
<% end %>
<script>
function onUpload(event) {
var name = event.fpfile.filename;
jQuery.ajax({
data: { "attachment[name]": name, "attachment[title]": url },
type: 'post',
url: "/attachments"
});
}
</script>
Route
post 'attachments/' => 'attachment#create'
Attachment Controller
def create
#attachment = current_user.attachments.build(attachment_params)
if #attachment.save
redirect_to attachments_path
else
render root_path
end
end
...
private
def attachment_params
params.require(:attachment).permit(:title, :user_id, :name)
end
I have browsed the relevant SO posts which is how I came to this implementation. However when I view my Attachment model in the rails console, name is nil. My data isn't getting passed properly. I am not sure if I am missing a step or am not routing properly. Any advice would be much appreciated.
Your controller super messy delete most of them and just keep it simple as like below. That one line will cover for you current_user.id and name.
def create
#attachment = current_user.attachments.build(attachment_params)
if #attachment.save
redirect_to attachments_path
else
render root_path
end
end
Also, meanwhile you have params permit
params.require(:attachment).permit(:name, :title, :user_id)
You have to pass data like this "attachment[name]", but you are passing it simple "name". Thats the reason getting Unpermitted parameter: name. And as #anonymousxxx mentioned remove :name part from routes!!!
Script
<script>
function onUpload(event) {
var name = event.fpfile.filename;
jQuery.ajax({
data: { "attachment[name]": name },
type: 'post',
url: "/attachments"
});
}
</script>
#attachment = current_user.attachments.build(attachment_params)
#attachment.user_id = current_user.id
You don't need to assgn current_user.id, because in first line you have build attachment with current_user
#foo = params[:name]
#attachment.name = #foo
That code is redundant, I think. Why you assign params[:name] to #foo ? If you have :name attribute in permit of attactment_params you don't need them.
Just
#attachment = current_user.attachments.build(attachment_params)
if #attacment.save
..
..
If you don't have it, you can use merge of params like this
#attachment = current_user.attachments.build(attachment_params.merge(name: params[:name]))
#attachment.save
..
..
Oh also you can merge in params method, rather than merge in action.
def attachment_params
params.require(:attachment).permit(:user_id).merge(name: params[:name])
end
Related
I am using the FileStack API and filepicker gem. I have the JavaScript below that parses a JSON browser response and sends it to the create action of the Attachment controller I have with Rails. I am trying to get it so that it can handle multiple files being uploaded and thus, multiple attachment models being created, and at the end, the form being submitted and the multiple models being saved. This code works with one file being uploaded, but I am trying to get it to work with multiple ones being dragged and dropped in. Any help would be greatly appreciated.
<%= simple_form_for #attachment, :html=> { id: 'file_stack_form' } do |f| %>
<%= f.filepicker_field :title, multiple: 'true', onchange: 'onUpload(event)' %>
<% end %>
<script>
function onUpload(event) {
var fileNumber = event.fpfiles.length;
var url = event.fpfile.url;
var name = event.fpfile.filename;
for (var i = 0; i < fileNumber; i++) {
jQuery.ajax({
data: { "attachment[name]": name, "attachment[title]" : url },
type: 'post',
url: "/attachments"
});
success: $('#file_stack_form').submit();
}
}
What type of :title have this field? When I uploaded images with carrierwave I used json type for my PostgreSQL database to upload multiple files.
Hi I'm making simple clone of game where is Waldo (Wally) with Rails + JS + Ajax. The idea is: player becomes 3 images and has to check where 3 characters (Waldo,Wilma,Wizard) are. After 3. image shows up field to submit name and after submit show high score list.
So far, i have code the mechanism for time (JS variable with setInterval), points (JS + AJAX + rails controller) but i can't code the action for submit name, points and time to model.
My view file:
<%= form_tag('/check/highscore', method: :post, format: :js, remote: true ) do %>
<%= text_field_tag 'name', 'enter your name' %>
<%= submit_tag 'submit' %>
<% end %>
<script type="text/javascript">
$.ajax({
url: "<%= highscore_check_index_path %>",
type: "POST",
format: "js",
dataType: "script",
data: { username: $(this).username, score: guessd, time: time },
success: function(data) {
$('send_highscore').append('data.username')
window.waldo.HS()
},
});
};
my controller file:
def highscore
#username = params[:name]
respond_to do |format|
format.js { render :layout => false }
format.html
end
end
def highscore2
record = Highscore.new
record.username = params[:username]
record.score = params[:score]
record.time = params[:time]
record.save
end
my highscore.js.erb
console.log("<%=j #username %>");
window.waldo.HS = function(username) {
$.ajax({
url: "<%= highscore2_check_index_path %>",
type: "POST",
format: "js",
data: { username: name, score: guessd, time: time },
success: function(data) {
console.log(data);
$('send_highscore').append('data.username')
}
});
});
I know, my oode is very bad quality, but i try to learn it now. Problem is that the highscore.js.erb is not executed although i see it in firebug rendered as a js script file.
The idea of highscore.js.erb is to mix ruby and js variables and together sent do highscore2 action in check controller and save to db.
js.erb is a bad way to learn.
Ruby on Rails can be used as a kind of poor mans Single Page Architecture by using remote: true and returning JS.erb responses. But all it really does it gives you enough rope to hang yourself.
Its a really bad way to learn ajax as it leads to poor code organisation and a very non RESTful application as it leads to a focus on procedures instead of resources.
I instead encourage you to try to learn ajax without mixing ERB and JS. Mixing server side and client side logic just makes everything much more complicated as you have to keep track of what is happening where.
Instead you might want to focus on setting up reusable javascript functions in app/assets/javascripts and fetching JSON data instead of javascript procedures.
This will teach you not only to do ajax in Rails but also how to do it with external API's and teach you valuable lessons about how to structure and organise code.
So lets start looking at how to refactor:
Highscores should be a resource.
Lets start by setting up the routes:
resources :highscores, only: [:create, :index, :show]
And the controller:
class HighscoresController < ApplicationController
respond_to :json, :html
def create
#highscore = Highscore.create(highscore_params)
respond_with #highscore
end
def index
#highscores = Highscore.order(score: :desc).all
respond_with #highscores
end
def show
#highscore = Highscore.find(params[:id])
respond_with #highscore
end
private
def highscore_params
params.require(:highscore).permit(:username, :time, :score)
end
end
Responders is a pretty awesome tool that makes it so that we don't have to do a bunch of boilerplate code when it comes to returning responses. Our create method will return 200 OK if it was successful and 422 BAD ENTITY or some other "bad" response code if it failed.
Some javascript scaffolding
Lets set up a little bit of scaffolding so that we can hook our javascript up to a specific controller and action without using inline script tags.
Open up layouts/application.html.erb and replace the <body> tag:
<%= content_tag :body, data: { action: action_name, controller: controller_name } do %>
<%= yield %>
<% end %>
Then add this little piece to applicaiton.js:
// Triggers events based on the current controller and action.
// Example:
// given the controller UsersController and the action index
// users:loaded
// users.index:loaded
$(document).on('page:change', function(){
var data = $('body').data();
$(this).trigger(data.controller + ":loaded")
.trigger(data.controller + "." + data.action + ":loaded");
});
Where is that ajax y'all was talkin' bout?
Lets say we have a list of highscores that we want to update either at regular intervals (polling) or when the user submits a highscore.
// app/assets/javascripts/highscores.js
function getHighscores(){
var $self = $('.highscores-list');
return $.getJSON('/highscores').done(function(highscores){
var elements = $.map(highscores, function(h){
return $('<li>').text([h.username, h.score].join(', '));
});
$self.empty().append(elements);
});
};
$(document).on('games:loaded highscores:loaded', function(){
// refresh highscores every 5 seconds
(function refresh(){
getHighscores().done(function(){
window.setTimeout(refresh, 5000);
});
}());
});
That last part is a bit hairy - its a recursive function that sets a new timer when the ajax call has completed. The reason we use this and not setInterval is that setInterval does not care if the previous call was completed or not.
Creating Highscores.
First lets russle up a form:
<%# app/views/highscores/_form.html.erb %>
<%= form_for(local_assigns[:highscore] || Highscore.new), class: 'highscore-form' do |f| %>
<div class="field">
<%= f.label :username %>
<%= f.text_field :username %>
</div>
<%= f.hidden_field :score %>
<%= f.hidden_field :time %>
<%= f.submit %>
<% end %>
And then lets spiff it up with some ajax goodness:
$(document).on('games:loaded highscores:loaded', function(){
$('#new_highscore').submit(function(e){
$.ajax(this.action, {
data: {
highscore: {
username: this.elements["highscore[username]"].value,
time: this.elements["highscore[time]"].value,
score: this.elements["highscore[score]"].value
}
},
method: 'POST',
dataType: 'JSON'
}).done(function(data, status, jqXHR){
getHighscores();
// todo notify user
});
return false; // prevent form from being submitted normally
});
});
I have a simple application that serves JSON through the API back end when creating a new user. I'm trying to add AJAX to the request so it doesn't reload the page when a new user is added. I'm really new to AJAX/API and have really no idea what needs to be done.
This is my Users API Controller:
class Api::UsersController < Api::BaseController
protect_from_forgery with: :null_session
def create
#user = User.create!(user_params)
respond_to do |format|
format.html { redirect_to users_list_path, notice: 'User was successfully created.' }
format.json { render json: #index }
format.js
end
end
private
def user_params
params.require(:user).permit(:name)
end
def query_params
params.permit(:name)
end
end
This is my index view(it should refresh with ajax call):
<% content_for :title do %>Users<% end %>
<h3>Here is the user list. Please click on one to see messages.</h3>
<br/>
<% User.all.each do |user| %>
<p><%= link_to String(user.id) + '. ' + user.name, messages_list_path(user_id: user.id) %></p>
<% end %>
<br/>
<%= form_tag("/api/users.json", method: "post") do %>
<%= label_tag(:add_user, "Add User:") %>
<%= text_field_tag("user[name]") %>
<%= submit_tag("Add") %>
<% end %>
The issue I am having is when I add a new user it takes me straight to http://localhost:3000/api/users.json which displays the JSON data. I need the app to basically stay on the same page where the form is and just simply update with AJAX.
I am really new to the building API's in rails as well as AJAX so any help would be great!
add remote: true to your form_tag to submit an ajax call.
see DOCS
form_tag("/api/users.json", method: "post", remote: true)
The problem is that the form automatically redirects you when submitted. You could put in a jQuery event listener to prevent that and then create the new user with an ajax call after, something like:
$('form').on('submit', submitForm.bind(this, event));
var submitForm = function (event) () {
event.preventDefault();
var user = $('form').serialize();
$.ajax({
url: "/api/users",
method: "POST",
data: user,
dataType: "json",
success: function(data){
this.handleSuccess(data);
}.bind(this)
});
};
Not sure if that's exactly what you're looking for, but hope it helps!
Running the following code shows that the Controller and the View are receiving and parsing the JSON appropriately, and returning HTML. (This can be seen through the alert in the call back function.)
However, what I want is for the browser to be redirected to /json_test/main, not to just receive the HTML in response.
Controller:
class JsonTestController < ApplicationController
def main
#postData = params[:scores]
end
def test
end
end
main.html.erb:
<h1>JsonTest#main</h1>
<p>Find me in app/views/json_test/main.html.erb</p>
<% #postData.each do |key, value| %>
<p><%= key %> : <%= value %></p>
<% end %>
test.html.erb:
<script type="text/javascript">
var postUrl = "<%= json_test_main_path %>";
var postJson = JSON.parse('{ "scores" : { "player1": 5, "player2": 6 } }');
$.get(postUrl, postJson, function(data, status, xhr) {
alert("Data: " + data + "\nStatus: " + status);
}, "html");
</script>
Any help would be appreciated.
What's the point of doing an AJAX request if your goal is to redirect the user unto another page?
There is no point.
Nonetheless, you can add a success callback in your $.get which redirects the browser.
.done(function() {
window.location.href = "http://stackoverflow.com";
})
I'm trying to update vote count via ajax once user has voted. My code works fine except for something which should be pretty basic, which is showing the new total number of votes.
My javascript has the following code:
var voteCount = "<%= #trip.total_up_votes %>";
...
$.ajax({
...
success: function() {
console.log("SAVED TO VOTES TABLE SUCCESSFULLY");
$('#voting_up').html(voteCount);
},
...
});
Once the vote link has been clicked, everything gets added to the table fine except it shows the new vote count as <%= #trip.total_up_votes %> i.e. as a string. The total_up_votes method simply counts the number of up votes from the votes table. This works fine when the page is first loaded or when it's refreshed.
I've tried escape_javascript and many other suggestions after trawling through the internet but I'm still stuck. Could someone help please?
EDIT:
As requested, my votes_controller does this:
def cast_vote()
#vote = Vote.where("user_id = ? AND trip_id = ?", current_user, params[:id]).first || Vote.new(:user_id => current_user)
#vote.vote_type = params[:vote_type]
#vote.user_id = params[:user_id]
#vote.trip_id = params[:id]
respond_to do |format|
#vote.save
format.html {redirect_to :back}
format.js
end
end
Have you tried dropping the quotes?
var voteCount = <%= #trip.total_up_votes %>;
...
$.ajax({
...
success: function() {
console.log("SAVED TO VOTES TABLE SUCCESSFULLY");
$('#voting_up').html(voteCount);
},
...
});
This is all overkill. You should use the ruby ajax helper "remote". use this:
<%= link_to "Vote up", vote_up_path, remote: true %>
The remote: true above will tell the browser to handle the link via ajax, so the page will not get reloaded. vote_up_path is just an example. Lets say vote_up_path takes you to the controller action votes#upvote, then in your votes views folder you should have a file called upvote.js.erb. In that file, have the following code:
var voteCount = <%= #trip.total_up_votes %>;
console.log("SAVED TO VOTES TABLE SUCCESSFULLY");
$('#voting_up').html(voteCount);